Skip to content

fix: hide PlayPage update icon for ignored updates#157

Merged
Nat3z merged 2 commits into
mainfrom
nat-openclaw/issue-155-issue-155-1780517503457-eoynnt
Jun 3, 2026
Merged

fix: hide PlayPage update icon for ignored updates#157
Nat3z merged 2 commits into
mainfrom
nat-openclaw/issue-155-issue-155-1780517503457-eoynnt

Conversation

@nat-openclaw

Copy link
Copy Markdown
Collaborator

Summary

  • Removes the secondary update icon on PlayPage when an update has been ignored/dismissed, so only actionable updates stand out.
  • Dismiss clearing on a newly detected version was already handled in addAppUpdate (stale dismiss entries are dropped when updateVersion changes).

Closes #155

Test plan

  • Ignore an update on PlayPage — no update icon or CTA should appear (Play still shows).
  • When addon reports a new updateVersion, update CTA and icon should return.
  • bun run typecheck in application/ passes locally.

Made with Cursor

@vercel

vercel Bot commented Jun 3, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
open-game-installer-web Ready Ready Preview, Comment Jun 3, 2026 8:20pm

@coderabbitai

coderabbitai Bot commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: a7144121-e261-4ec2-951e-731c5fc991cb

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch nat-openclaw/issue-155-issue-155-1780517503457-eoynnt

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Nat3z

Nat3z commented Jun 3, 2026

Copy link
Copy Markdown
Owner

@nat-openclaw undo what you did for playpage. i meant the libraryview with the same context.

@nat-openclaw

nat-openclaw commented Jun 3, 2026

Copy link
Copy Markdown
Collaborator Author
▶️ Actions Performed
  • Restore PlayPage dismissed-state update icon block — Restoring removed PlayPage update icon block
  • Hide LibraryView update badges when update is dismissed
  • Run application typecheck — Running bun run typecheck in application/
  • Commit and push to PR branch — Committing and pushing to PR branch

Revert the PlayPage change and apply dismiss-aware update badge hiding
in LibraryView for Recently Played and All Games tiles.
@nat-openclaw

Copy link
Copy Markdown
Collaborator Author

Reverted the PlayPage change and moved the dismiss-aware update handling over to LibraryView instead.

  • PlayPage is back to showing the secondary green update icon as before
  • LibraryView now hides the yellow update badges for ignored update versions
  • UMU migration badges are unchanged

I ran bun run typecheck in application/ and it passed. Pushed as 05c2f0c.

@Nat3z Nat3z left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

lgtm

@Nat3z Nat3z merged commit 2c3d11a into main Jun 3, 2026
7 checks passed
@Nat3z Nat3z deleted the nat-openclaw/issue-155-issue-155-1780517503457-eoynnt branch June 3, 2026 20:26
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.

Have Ignored Update Games Not Show The Update Icon in the PlayPage

2 participants