Skip to content

[Omni] Refresh badge after deleting popup assets#85

Merged
olgahaha merged 1 commit into
mainfrom
omni/e42f8a19/proofsnap-extension
May 6, 2026
Merged

[Omni] Refresh badge after deleting popup assets#85
olgahaha merged 1 commit into
mainfrom
omni/e42f8a19/proofsnap-extension

Conversation

@numbers-official
Copy link
Copy Markdown
Collaborator

Summary

This PR ensures the extension badge stays in sync after popup assets are deleted. It adds a background message path for refreshing badge state and has the popup trigger it after a successful delete, preventing stale badge counts or indicators.

Changes

  • Add badge refresh handling in src/background/service-worker.ts for popup-initiated refresh requests.
  • Update src/popup/popup.tsx to request a badge refresh after deleting popup assets.
  • Extend shared message types in src/types/index.ts to include the new badge refresh message.

4 files changed, +28 additions, -2 deletions

File changes
.omni/e42f8a19-untitled/memory.md |  3 ++-
 src/background/service-worker.ts  | 11 +++++++++++
 src/popup/popup.tsx               | 13 +++++++++++++
 src/types/index.ts                |  3 ++-

Generated by Olga Shen with Omni

@olgahaha olgahaha merged commit 21cef40 into main May 6, 2026
1 check passed
@olgahaha olgahaha deleted the omni/e42f8a19/proofsnap-extension branch May 6, 2026 07:38
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