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

Maintain the Privacy Policy help notice location #11

Merged
merged 5 commits into from
Nov 26, 2018
Merged

Maintain the Privacy Policy help notice location #11

merged 5 commits into from
Nov 26, 2018

Conversation

desrosj
Copy link
Contributor

@desrosj desrosj commented Nov 19, 2018

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.

@azaozz
Copy link
Contributor

azaozz commented Nov 23, 2018

@desrosj updated this after all changes in beta-1. The remove_action/add_action have to run only when $settings['replace'] || self::is_classic( $post_id ). Could you see if it works as expected?

@desrosj
Copy link
Contributor Author

desrosj commented Nov 26, 2018

Tested this with the following conditions:

  • 5.0 RC1 with Classic Editor activated (no Gutenberg plugin).
  • 5.0 RC1 with both Classic Editor and Gutenberg activated.
  • WordPress 4.9.8 with Gutenberg plugin and Classic Editor active.
  • WordPress 4.9.8 with only the Classic Editor active (no Gutenberg plugin).

All scenarios correctly restored/preserved the notice location.

@azaozz azaozz merged commit 7076276 into WordPress:master Nov 26, 2018
@desrosj desrosj deleted the privacy-policy-help-notice branch November 26, 2018 20:21
@youknowriad
Copy link

I wonder if we need to revert this because the admin notices support was removed from RC2

@desrosj
Copy link
Contributor Author

desrosj commented Dec 4, 2018

@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:

  • Reverted of r43920 in 5.0.1, and the user does not update the Classic Editor plugin
  • Revert this change in Classic Editor and user stays on a version of core less than the one with a revert of 443920.

I have this on my list to follow up on with a new issue when it becomes clearer.

@youknowriad
Copy link

Thanks for the clarification @desrosj just wanted to make sure it was not something we missed :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants