Skip to content

Updated code to use wp_admin_notice.#1080

Merged
mauteri merged 1 commit intodevelopfrom
GP-896
Apr 27, 2025
Merged

Updated code to use wp_admin_notice.#1080
mauteri merged 1 commit intodevelopfrom
GP-896

Conversation

@mauteri
Copy link
Copy Markdown
Contributor

@mauteri mauteri commented Apr 27, 2025

Description of the Change

  • Update spots to use wp_admin_notice.
  • Remove site check functionality to see if registration is on. It broke convention and will not be needed in 0.33.0 which introduces Open RSVP.

Closes #896

How to test the Change

Changelog Entry

Changed - Existing functionality
Removed - Feature

Credits

Props @mauteri, @carstingaxion

Checklist:

  • I agree to follow this project's Code of Conduct.
  • I have updated the documentation accordingly.
  • I have added tests to cover my change.
  • All new and existing tests pass.

… longer be needed in 0.33.0 and broke conventions.
@github-actions
Copy link
Copy Markdown
Contributor

Preview changes with Playground

You can preview the recent changes for PR#1080 with the following PHP versions:

PHP Version 8.3

PHP Version 7.4

Download .zip with build changes

Made with 💙 from GatherPress & a little bit of WordPress Playground. Changes will not persist between sessions.

@mauteri mauteri marked this pull request as ready for review April 27, 2025 21:20
@mauteri mauteri merged commit 2a22eb1 into develop Apr 27, 2025
14 checks passed
@mauteri mauteri deleted the GP-896 branch April 27, 2025 21:20
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.

Make use of wp_admin_notice where possible

1 participant