Skip to content

Sync update notification message after each relevant action - #3410

Merged
Salvialf merged 3 commits into
developfrom
feat/update-message-sync
Jul 22, 2026
Merged

Sync update notification message after each relevant action#3410
Salvialf merged 3 commits into
developfrom
feat/update-message-sync

Conversation

@Salvialf

@Salvialf Salvialf commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Fix message::removeAll('update', 'newUpdate') in install/update.php that never deleted anything: message::save() randomizes the logicalId when it equals 'newUpdate', so no message ever had that logicalId in the database.
  • Add update::refreshUpdateMessage() as a central method to manage the update notification message lifecycle: removes stale messages when the pending list changes, leverages message::save() native dedup to avoid spurious toasts when the list is unchanged, and cleans up entirely when no updates are pending.
  • Call refreshUpdateMessage() after every operation that can change update statuses: jeedom::cronHourly() and the update, checkUpdate, checkAllUpdate actions in update.ajax.php.

@Salvialf Salvialf added the changelog-feat Use to generate release notes / changelog. To be apply on PR. Use it only for core feature label Jul 1, 2026
@Salvialf
Salvialf merged commit b894898 into develop Jul 22, 2026
9 checks passed
@Salvialf
Salvialf deleted the feat/update-message-sync branch July 22, 2026 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog-feat Use to generate release notes / changelog. To be apply on PR. Use it only for core feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants