-
Notifications
You must be signed in to change notification settings - Fork 81
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
Maintain the Privacy Policy help notice location #11
Conversation
@desrosj updated this after all changes in beta-1. The remove_action/add_action have to run only when |
Tested this with the following conditions:
All scenarios correctly restored/preserved the notice location. |
I wonder if we need to revert this because the admin notices support was removed from RC2 |
@youknowriad This can't be reverted until r43920 is reverted in WordPress Core. Even then, It will be tricky and I think it's best to leave this in here in some form for backward compatibility. Scenarios I think of are:
I have this on my list to follow up on with a new issue when it becomes clearer. |
Thanks for the clarification @desrosj just wanted to make sure it was not something we missed :) |
When using the classic editor, the Privacy Policy help notice location in 4.9.x (directly under the post title with the
edit_form_after_title
action hook) should be maintained.Related: WordPress/gutenberg#11999, WordPress/gutenberg#11604, Trac-45057.