Skip to content

Update EVA logo and remove PWA install prompt#699

Merged
ngoiyaeric merged 1 commit into
mainfrom
update-logo-and-remove-pwa-prompt-8519713623494986006
Jun 27, 2026
Merged

Update EVA logo and remove PWA install prompt#699
ngoiyaeric merged 1 commit into
mainfrom
update-logo-and-remove-pwa-prompt-8519713623494986006

Conversation

@ngoiyaeric

@ngoiyaeric ngoiyaeric commented Jun 27, 2026

Copy link
Copy Markdown
Collaborator

This change updates the EVA logo in the Usage view and removes the PWA install prompt from the application.

Key changes:

  • Replaced public/images/eva-logo.png with the new logo provided by the user.
  • Removed InstallPrompt from app/layout.tsx.
  • Deleted the components/install-prompt.tsx file as it is no longer used.

PR created automatically by Jules for task 8519713623494986006 started by @ngoiyaeric

Summary by CodeRabbit

  • New Features
    • Replaced the app install prompt with a software update notification, helping users stay informed about available updates.
  • Chores
    • Updated a test tool dependency to the latest patch release.

Co-authored-by: ngoiyaeric <115367894+ngoiyaeric@users.noreply.github.com>
@google-labs-jules

Copy link
Copy Markdown
Contributor

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

@vercel

vercel Bot commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
qcx Building Building Preview, Comment Jun 27, 2026 4:50pm

@qodo-code-review

Copy link
Copy Markdown
Contributor

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@coderabbitai

coderabbitai Bot commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 46ebdd23-2c06-4b71-bec1-f013f2af6e92

📥 Commits

Reviewing files that changed from the base of the PR and between 7ea6e5e and 0ca6100.

⛔ Files ignored due to path filters (2)
  • bun.lock is excluded by !**/*.lock
  • public/images/eva-logo.png is excluded by !**/*.png
📒 Files selected for processing (3)
  • app/layout.tsx
  • components/install-prompt.tsx
  • package.json
💤 Files with no reviewable changes (2)
  • app/layout.tsx
  • components/install-prompt.tsx
📜 Recent review details
🔇 Additional comments (1)
package.json (1)

109-109: LGTM!


Walkthrough

Removes the InstallPrompt component and its file (components/install-prompt.tsx), replacing it with SWUpdateNotification in app/layout.tsx. Also bumps @playwright/test from ^1.60.0 to ^1.61.1 in devDependencies.

Replace InstallPrompt with SWUpdateNotification

Layer / File(s) Summary
Swap InstallPrompt for SWUpdateNotification in layout
app/layout.tsx, components/install-prompt.tsx
InstallPrompt import and render removed; SWUpdateNotification import added and rendered at the same location in the MapLoadingProvider tree. components/install-prompt.tsx (53 lines) deleted entirely.
Bump @playwright/test
package.json
@playwright/test devDependency updated from ^1.60.0 to ^1.61.1.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • QueueLab/QCX#686: Added SWUpdateNotification and InstallPrompt wiring to app/layout.tsx, directly preceding the removal of InstallPrompt in this PR.
  • QueueLab/QCX#697: Modified components/install-prompt.tsx with a useIsStandalone hook — the same file deleted in this PR.

Poem

🐇 Hop hop, the prompt is gone,
No more "Install App" button to click on!
SWUpdateNotification takes the stage,
A fresh new component, the new age.
The rabbit cheers with a little dance~ 🎉

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main changes: updating the EVA logo and removing the PWA install prompt.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch update-logo-and-remove-pwa-prompt-8519713623494986006

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.

@charliecreates charliecreates Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Reviewed the latest changes, and I don't have actionable feedback to address.

@ngoiyaeric ngoiyaeric merged commit 79065e7 into main Jun 27, 2026
4 of 5 checks passed
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