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.
Add API documentation to the Route process plugin.
Comment | File | Size | Author |
---|---|---|---|
#9 | 2845487-9.patch | 1.51 KB | jofitz |
#9 | interdiff-6-9.txt | 880 bytes | jofitz |
#6 | 2845487-6.patch | 1.63 KB | jofitz |
#6 | interdiff-2-6.txt | 1.6 KB | jofitz |
#2 | 2845487-1.patch | 927 bytes | quietone |
Comments
Comment #2
quietone CreditAttribution: quietone as a volunteer commentedComment #3
phenaproximaSelf-assigning for review.
Comment #5
phenaproximaThis seems incomplete :)
The first sentence needs to end with a colon, and both items in the input value need to be explained.
We need an explanation of what this example does. And maybe another example as well, if it would be helpful.
Comment #6
jofitz CreditAttribution: jofitz at ComputerMinds commentedChanges in response to code review:
Comment #7
phenaproximaSelf-assigning for review.
Comment #8
phenaproximaThis effectively repeats the short description. We're going to need to either explain this a lot more thoroughly (i.e., by linking to the main documentation for the D8 routing system) or remove it altogether.
Can we add an example showing what will be returned if you pass an internal path like /admin/content as the source value?
Comment #9
jofitz CreditAttribution: jofitz at ComputerMinds commentedComment #10
phenaproximaI like it.
Comment #11
alexpottCommitted and pushed 2cfd4d7 to 8.4.x and 63ab9aa to 8.3.x. Thanks!