Skip to content

Hide Admin Notice about alpha-plugin, also if installed via the branch-named zip#808

Merged
mauteri merged 4 commits intoGatherPress:mainfrom
carstingaxion:fix/807
Aug 16, 2024
Merged

Hide Admin Notice about alpha-plugin, also if installed via the branch-named zip#808
mauteri merged 4 commits intoGatherPress:mainfrom
carstingaxion:fix/807

Conversation

@carstingaxion
Copy link
Copy Markdown
Collaborator

@carstingaxion carstingaxion commented Aug 16, 2024

While a plugin from w.org would have a unigue slug to check against,
this is different for .zips downloaded from github.com.

A download via "<> Code ⏷" > "Download ZIP" will include the branch in the file name,
while a download via "Releases" will provide a zip without a branch name attached.

Description of the Change

This PR makes sure, that GatherPress checks against both possible slugs inside the condition to whether or not to show the admin notice about the alpha-plugin.

Closes #807

How to test the Change

Changelog Entry

Fixed - Hide Admin Notice about alpha-plugin, also if installed via the branch-named zip

Credits

Props @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.

@carstingaxion carstingaxion requested a review from mauteri August 16, 2024 13:58
@github-actions
Copy link
Copy Markdown
Contributor

Preview changes with Playground

You can preview the least recent changes for PR#808 by following one of the links below:

⚠️ Note: The preview sites are created using WordPress Playground. You can add content, edit settings, and test the themes as you would on a real site, but please note that changes are not saved between sessions.

@carstingaxion
Copy link
Copy Markdown
Collaborator Author

Friendly ping @mauteri ;)

@mauteri mauteri merged commit 4c5b27d into GatherPress:main Aug 16, 2024
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.

seeing The GatherPress Alpha plugin notice

2 participants