Skip to content

Conversation

@dkotter
Copy link
Collaborator

@dkotter dkotter commented Apr 14, 2025

Description of the Change

Instead of relying on the normal plugin activation notice, this PR adds in a custom notice that prompts users to go to the settings page to connect their account:

Plugin activation notice

We may need to update the logo used here, I just pulled this from a quick search. And can adjust the styling more if desired, left this pretty basic for now.

In addition to this change, also added the logo to the settings page and made a few adjustments there. Also add some additional styling to the error state in the block.

How to test the Change

Checkout this PR then activate the plugin. Ensure the notification shows and links to the correct place

Changelog Entry

Added - Custom admin notification after plugin activation
Added - Show logo on the settings page

Credits

Props @dkotter

Checklist:

@dkotter dkotter added this to the 1.0.0 milestone Apr 14, 2025
@dkotter dkotter self-assigned this Apr 14, 2025
@dkotter dkotter requested a review from faisal-alvi April 14, 2025 19:48
@github-actions github-actions bot added the needs:refresh This requires a refreshed PR to resolve. label Apr 15, 2025
@github-actions github-actions bot added needs:code-review This requires code review. and removed needs:refresh This requires a refreshed PR to resolve. labels Apr 15, 2025
@faisal-alvi faisal-alvi merged commit c7fdbd9 into develop Apr 15, 2025
7 of 8 checks passed
@faisal-alvi faisal-alvi deleted the add/plugin-notification branch April 15, 2025 11:27
@github-project-automation github-project-automation bot moved this to Backlog in 10up Jobber Apr 16, 2025
@jeffpaul jeffpaul moved this from Backlog to Done in 10up Jobber Apr 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs:code-review This requires code review.

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants