-
Notifications
You must be signed in to change notification settings - Fork 138
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Translate: Add site tour items #172
Conversation
wordpress.org/public_html/wp-content/plugins/wporg-gp-customizations/inc/class-plugin.php
Show resolved
Hide resolved
175b517
to
9c8430e
Compare
It works, I just wondered if we could also just enqueue one style and one script since the dependency system would automatically load the required ones? |
True, I have updated the PR, thank you! |
Looks good to merge after GlotPress/GlotPress#1632 |
Although, there was this one hiccup with the meta toolbar step in the tour, were you able to reproduce? Would be good to know if it is a bug in this PR or in GlotPress/GlotPress#1632 |
Yes, I'm able to reproduce and I'm taking a look |
I have confirmed that it's a bug on GlotPress/GlotPress#1632 |
@trymebytes can we close this PR? |
Done, thanks! |
Add tour items for translate.wordpress.org, which is loaded only if the correct GET parameter is appended to the page url.
This PR adds a site tour to translate.wordpress.org by adding this filter below;
To test this PR;
site-tour
branch and rungit pull
to pull latest updates.site_tour=test
to the end of your url for example, to view the site tour on the Events Manager project; https://translate.wordpress.org/projects/wp-plugins/events-manager/dev/art-xemoji/default/?filters%5Bstatus%5D=untranslated&site_tour=test