Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
We have a use case where the translator service wants to filter out content based on a set of criteria before the job is created. We couldn't find an appropriate place to do that.
Proposed resolution
Add an event and invoke it in \Drupal\tmgmt\ContinuousManager::addItem
Remaining tasks
User interface changes
API changes
Data model changes
Comment | File | Size | Author |
---|---|---|---|
#5 | 2933917-additem-event-5-interdiff.txt | 15.33 KB | Berdir |
#5 | 2933917-additem-event-5.patch | 14.95 KB | Berdir |
| |||
#3 | 2933917-additem-event-2.patch | 10.13 KB | Primsi |
| |||
#3 | 2933917-additem-event-2.interdiff.txt | 3.58 KB | Primsi |
#2 | 2933917-additem-event.patch | 6.56 KB | Primsi |
|
Comments
Comment #2
Primsi CreditAttribution: Primsi at MD Systems GmbH for Translations.com commentedInitial patch, no tests yet.
Comment #3
Primsi CreditAttribution: Primsi at MD Systems GmbH for Translations.com commentedStarting with tests, not sure if this is the direction we want to go.
Comment #4
BerdirI think it makes more sense to name that class ContinuousEvents.
I'll try to improve the docs a bit here.
...Interface
.. ID
.
whether
improve this a bit as well.
continuous events
I think it's better to avoid the UI and just check if we have any jobs here, and then create a node with a different label that will be added.
.
Comment #5
BerdirCleanup and moved the test to the existing kernel test coverage, just requires a few additional lines then.
interdiff is very big but it's mostly just comment changes and the moving the test.
Comment #6
BerdirCommitted.