- ๐บ๐ธUnited States smustgrave
This issue is being reviewed by the kind folks in Slack, #needs-review-queue-initiative. We are working to keep the size of Needs Review queue [2700+ issues] to around 400 (1 month or less), following Review a patch or merge request โ as a guide.
Removing the tests tag as they were added in #18
Will need a change record to announce the hook.
- First commit to issue fork.
- ๐ฎ๐ณIndia bhanu951
Patch in #25 doesnt contain all the changes present in patch from #18
Changes to file core/modules/content_translation/content_translation.api.php is missing in #25
- ๐ฎ๐นItaly apaderno Brescia, ๐ฎ๐น
I have hidden the patch in comment #25.
- ๐ง๐ชBelgium kevinvb
Maybe we should close this one in favor of Content translation access control ๐ Content translation access control Needs work
I've succesfully used that one to be able to provide entity access conditions on create translation, update translation or delete translation.
Without the need of an additional hook. - ๐ช๐ธSpain eduardo morales alberti Spain, ๐ช๐บ
eduardo morales alberti โ made their first commit to this issueโs fork.
- Merge request !13044Issue #2918354 by marckwee, mohit_aghera, seanb: Add hook_entity_translation_access() โ (Open) created by eduardo morales alberti
- ๐ช๐ธSpain eduardo morales alberti Spain, ๐ช๐บ
Create an MR with the patch from comment #18, on drupal 11.x the file core/modules/content_translation/tests/modules/content_translation_test/content_translation_test.module is not available.
- ๐ช๐ธSpain eduardo morales alberti Spain, ๐ช๐บ
As comment #30 said, we should decide what is the best option to control the access.
The Needs Review Queue Bot โ tested this issue. It fails the Drupal core commit checks. Therefore, this issue status is now "Needs work".
This does not mean that the patch necessarily needs to be re-rolled or the MR rebased. Read the Issue Summary, the issue tags and the latest discussion here to determine what needs to be done.
Consult the Drupal Contributor Guide โ to find step-by-step guides for working with issues.