Translations created in a workspace are not shown in the admin content overview

Here's the fix for this. Marking as NW because it needs test coverage.

Thanks for getting that going Andrei! How does this tweak look?

I noticed that if I create a translation > delete it > recreate it, I get an invalid translation error cause it already exists.

Without the docroot... 😅

🫠

@amateescu Do we need that $default_revision->setUnpublished(); part? It doesn't seem like it, but I'm not quite sure. 🤔

We just noticed that when we add a translation the Live version is disappearing. 😅

Do we need that $default_revision->setUnpublished(); part?

Yep, we do, but we need to set on the proper object, which is the translation we just created :)

While writing test coverage for this bug, I found that hook_entity_translation_insert() is a better choice for handling this situation.

The Needs Review Queue Bot → tested this issue.

While you are making the above changes, we recommend that you convert this patch to a merge request → . Merge requests are preferred over patches. Be sure to hide the old patch files as well. (Converting an issue to a merge request without other contributions to the issue will not receive credit.)

Small nitpicky

But 1 question about the return of hook_entity_translation_insert

Addressed the feedback :)

Feedback appears to be addressed!

Thanks!

One comment on the MR.

Fixed the last comment :)

Committed/pushed to 11.x and cherry-picked to 10.3.x, thanks!

Automatically closed - issue fixed for 2 weeks with no activity.