Skip to content

Comments

Tweaks for the "Improve Plugin" PR#4

Merged
jdevalk merged 2 commits intojdv/improve-pluginfrom
jdv/improve-plugin-tweaks
Oct 10, 2025
Merged

Tweaks for the "Improve Plugin" PR#4
jdevalk merged 2 commits intojdv/improve-pluginfrom
jdv/improve-plugin-tweaks

Conversation

@ilicfilip
Copy link
Contributor

This PR implements following changes:

  • Block name was set to my-plugin/brand-asset, so that is changed. It will be a "breaking" change since old block will no longer be recognized by Gutenberg, but that was already the case when the block was renamed from my-plugin/color-scheme to my-plugin/brand-assets
  • init_hooks method was called in the constructor of the Settings and Front End classes, but that makes hooks being added multiple times if class is instantiated more than once - which was done here (so hooks were added twice on front end). I changed init_hooks methods to be public, so they can be called explicitly (or we can convert classes to singleton pattern).

@ilicfilip ilicfilip requested a review from jdevalk October 10, 2025 07:33
@github-actions
Copy link
Contributor

Test on Playground
Test this pull request on the Playground
or download the zip

@jdevalk jdevalk merged commit f5be22f into jdv/improve-plugin Oct 10, 2025
5 checks passed
@jdevalk jdevalk deleted the jdv/improve-plugin-tweaks branch October 10, 2025 18:23
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.

2 participants