Currently converting an entity type to revisionable does not work for an entity with an integer ID.
SqlContentEntityStorageSchemaConverter::convertToRevisionable() (to be precise: SqlContentEntityStorageSchemaConverter::copyData() contains the following:
// Set the revision ID to be same as the entity ID.
$entity->set($revision_id_key, $entity_id);
The revision ID is always an integer, but the entity ID may be a string, in which case this will break.
?
Active
11.0 🔥
The change is currently missing an automated test that fails when run with the original code, and succeeds when the bug has been fixed.
Not all content is available!
It's likely this issue predates Contrib.social: some issue and comment data are missing.
Ran into this issue myself as well, I've added a small patch that helped me with making my entity type (that uses a machine-name as id) revisionable
I haven't added any tests, just a POC that seems to work good enough for me to continue :)