Skip to content
This repository has been archived by the owner on Jul 28, 2023. It is now read-only.

馃帹 Fix context not updated on prop change #90

Merged
merged 1 commit into from
Oct 21, 2022
Merged

馃帹 Fix context not updated on prop change #90

merged 1 commit into from
Oct 21, 2022

Conversation

luisherranz
Copy link
Member

I did another small fix for the wp-context that was not being updated whenever the prop or the attribute changed.

https://www.loom.com/share/f8806d4cc9ad4580984eff98cb2dfcd9

@luisherranz luisherranz self-assigned this Oct 21, 2022
@luisherranz luisherranz mentioned this pull request Oct 21, 2022
Closed
23 tasks
SantosGuillamot
SantosGuillamot approved these changes Oct 21, 2022
View reviewed changes
Copy link
Contributor

@SantosGuillamot SantosGuillamot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great 馃檪

@luisherranz
Copy link
Member Author

Thanks, Mario! 馃檪

@luisherranz luisherranz merged commit 0f90f67 into main-wp-directives-plugin Oct 21, 2022
@luisherranz luisherranz deleted the fix-context-not-updated-on-prop-change branch October 21, 2022 14:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@SantosGuillamot SantosGuillamot SantosGuillamot approved these changes

@DAreRodz DAreRodz Awaiting requested review from DAreRodz

Assignees

@luisherranz luisherranz

Labels
馃帹 WordPress Directives Plugin
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@luisherranz @SantosGuillamot