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.
When we removed the duplicate links for local actions, this changed the route for views administration. However, the Views UI tour was not updated.
Attached resolves the issue.
Comment | File | Size | Author |
---|---|---|---|
#7 | views-ui-tour-2040369-7.patch | 1.88 KB | xjm |
#7 | interdiff-tour.txt | 916 bytes | xjm |
views-ui-tour.patch | 1.87 KB | xjm | |
views-ui-tour-FAIL.patch | 1.41 KB | xjm | |
Comments
Comment #1
xjmClosed #2027469: View UI Tour is not working and had a sandbox update. as a duplicate of this issue.
Comment #3
tim.plunkettviews-ui-tour.patch queued for re-testing.
Comment #4
larowlanTesting for this being worked on in #2028535: Provide a TourTestBase class for use by core and contrib modules
Comment #5
xjm@larowlan: For views, for now, it's probably better to implement Views' base test class (as I have) since this provides a test view pretty easily. This fixes a regression and tests are a gate.
We can add more thorough tests later based on the new base class. I'll comment on the other issue.
Comment #6
dawehnerGreat a patch!
These two lines seems to disagree, but I agree it should be part of views_ui (that is the place of the config file).
Needs an empty line.
Comment #7
xjmThanks @dawehner! Fixed in attached.
Comment #8
dawehnerThank you.
Comment #9
alexpottCommitted f92f05c and pushed to 8.x. Thanks!