Skip to content
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

Guided Tours: hide “Editor Insert Menu” tour behind a dev-only feature flag #10707

Closed
wants to merge 1 commit into from

Conversation

Copons
Copy link
Contributor

@Copons Copons commented Jan 17, 2017

Closes #10568

This PR removes the Editor Insert Menu guided tour from anywhere except development as it's not reliable enough, and it keeps reappearing when it's not supposed to anymore.

I've decided to keep it in development mostly because it's behaviour is more consistent there, and to have a chance of finding out what's wrong with it.

@Copons Copons added Guided Tours [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. [Type] Bug labels Jan 17, 2017
@Copons Copons requested review from lsinger and mcsf January 17, 2017 18:41
@matticbot
Copy link
Contributor

@Copons
Copy link
Contributor Author

Copons commented Jan 17, 2017

Nevermind, I've been beaten for a couple of minutes!

@Copons Copons closed this Jan 17, 2017
@Copons Copons deleted the remove/editor-insert-menu-tour branch January 17, 2017 18:44
@lancewillett lancewillett removed the [Status] Needs Review The PR is ready for review. This also triggers e2e canary tests and wp-desktop tests automatically. label Jan 18, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants