Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented Jun 26, 2025

Description

Prevents the modal from popping up when running roomote locally. We should figure out a better long-term solution though.


Important

Update lastShownAnnouncementId in EVALS_SETTINGS to prevent modal pop-up when running roomote locally.

  • Behavior:
    • Update lastShownAnnouncementId in EVALS_SETTINGS in global-settings.ts from "may-29-2025-3-19" to "jun-17-2025-3-21".
    • Prevents modal pop-up when running roomote locally.

This description was created by Ellipsis for d2002bc. You can customize this summary. It will automatically update as commits are pushed.

@delve-auditor
Copy link

delve-auditor bot commented Jun 26, 2025

No security or compliance issues detected. Reviewed everything up to d2002bc.

Security Overview
  • 🔎 Scanned files: 1 changed file(s)
Detected Code Changes
Change Type Relevant files
Configuration changes ► global-settings.ts
    Update lastShownAnnouncementId

Reply to this PR with @delve-auditor followed by a description of what change you want and we'll auto-submit a change to this PR to implement it.

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jun 26, 2025
@hannesrudolph hannesrudolph added the Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. label Jun 26, 2025
@daniel-lxs daniel-lxs moved this from Triage to PR [Needs Review] in Roo Code Roadmap Jun 26, 2025
@hannesrudolph hannesrudolph added PR - Needs Review and removed Issue/PR - Triage New issue. Needs quick review to confirm validity and assign labels. labels Jun 26, 2025
Copy link
Collaborator

@mrubens mrubens left a comment

Choose a reason for hiding this comment

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

Are we going to need to update this every time we change what's in clineprovider? Any thoughts on how we could share this value between them?

@cte cte closed this Jul 11, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jul 11, 2025
@github-project-automation github-project-automation bot moved this from PR [Needs Review] to Done in Roo Code Roadmap Jul 11, 2025
@cte cte deleted the cte/update-evals-settings branch July 11, 2025 19:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR - Needs Review size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants