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.
Comment | File | Size | Author |
---|---|---|---|
#14 | 3127449-14.patch | 11.12 KB | Rajab Natshah |
#2 | 3127449-2.patch | 6.71 KB | Suresh Prabhu Parkala |
Comments
Comment #2
Suresh Prabhu Parkala CreditAttribution: Suresh Prabhu Parkala at Specbee commentedPlease review.
Comment #3
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedDrupal coding standard and Drupal Practice check should pass
PHPCS, PHPCBF for both --standard=Drupal and --standard=DrupalPractice
Check Tour Builder Drupal coding standard
Check Tour Builder Drupal Practice
Comment #4
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #5
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #6
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #7
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedPlease if you can spare time to commit this issue to the dev branch that will be nice.
A release after that will have a support for Drupal 9 build.
as this issue is a blocker issue for
#3122672: Start a 9.0.x branch for Varbase and Varbase Project to integrate with Drupal 9
Real physical testing with Drupal 9 is important as Drupal 9.0.4 went out
Thanks for having time to work on this nice and very useful module.
Comment #8
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #9
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #10
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #11
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #12
clemens.tolboom@RajabNatshah thanks for spending time on this.
You're the first who adds the commands ran which is how it should be :-)
I think I saw removed comments which should not be removed as I want to restore the patch workflow one day.
What non automated changes do I have to make as you changed code between #4 and #9 ... an innerdiff would be handy.
Comment #13
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedThanks a lot Clemens for having time to follow on this issue.
Noted;
About the Batch workflow.
I had a look at the code for the Layout Builder Block Sanitizer
Learned a lot from them
The Batch class and services, and route
A new issue for that for sure.
I wanted to create the Batch issue but I had no extra time.
Thank you :)
Comment #14
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedChanged the patch based on Clemens last review.
Thank you :)
Comment #15
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #16
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedComment #17
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedI hope that you do have time to commit this issue
Thank you for maintaining this module.
a Release after this issue will have the module work under Drupal 9
Comment #18
Rajab Natshah CreditAttribution: Rajab Natshah at Vardot commentedMy hope to commit this issue and a new tag release
I had to #3176580: [TEMP] Remove [Tour Builder] module from the composer on the 9.0.x branch
This module is very helpful to work on building tours in projects
I like to have it work under Drupal 9 websites
#3122672: Start a 9.0.x branch for Varbase and Varbase Project to integrate with Drupal 9
Comment #20
clemens.tolboomI skipped the following changes
That was in for a reason I cannot remember so need to test first
This made installing tour_builder not updating the Action buttons on Tour UI page so is that still the case. Needs testing
PHP Storm complained about this ... why?
Comment #21
clemens.tolboomWith
composer require drupal/tour_builder:1.x-dev
I was able to test manually.Export gives an error on /admin/config/user-interface/tour/manage/views-ui/export
Not sure that was already an issue but added #3185215: Tour export broken
Comment #23
clemens.tolboomI call this one fixed now.
Thanks for your patience :-)