Skip to content

Fix config change hook initialization and add test - #87

Merged
novalisdenahi merged 2 commits into
masterfrom
set-init-config-change-hook-fix
Oct 14, 2026
Merged

novalisdenahi merged 2 commits into
masterfrom
set-init-config-change-hook-fix

Conversation

@novalisdenahi

@novalisdenahi novalisdenahi commented Oct 12, 2026 •

Copy link
Copy Markdown
Contributor

Describe the purpose of your pull request

  • Initialize set cachedEntry only if conditions are met and invoke the OnConfigChanged hook

Related issues (only if applicable)

https://trello.com/c/rxxuNh5o
https://trello.com/c/GAj9Tit2

How to test? (only if applicable)

  • What part of the application was affected by the changes? What should be tested?

Security (only if applicable)

  • Describe any potential risks that could affect the component in terms of security.

Requirement checklist (only if applicable)

  • I have covered the applied changes with automated tests.
  • I have executed the full automated test set against my changes.
  • I have validated my changes against all supported platform versions.
  • I have read and accepted the contribution agreement.

@novalisdenahi
novalisdenahi requested a review from a team as a code owner October 12, 2026 14:33
@sonarqubecloud

Copy link
Copy Markdown

@novalisdenahi
novalisdenahi merged commit e3a9b4b into master Oct 14, 2026
10 checks passed
@novalisdenahi
novalisdenahi deleted the set-init-config-change-hook-fix branch October 14, 2026 10:46
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