Skip to content

docs: Deep links docs update#2398

Merged
charlesvien merged 2 commits into
mainfrom
05-27-docs_update
May 28, 2026
Merged

docs: Deep links docs update#2398
charlesvien merged 2 commits into
mainfrom
05-27-docs_update

Conversation

@charlesvien
Copy link
Copy Markdown
Member

@charlesvien charlesvien commented May 27, 2026

Problem

Nothing to see here, just your STANDARD DOCS UPDATE!

Changes

  • Updated deep links docs to include missing deep link paths

How did you test this?

With Joy

Publish to changelog?

no

@charlesvien charlesvien changed the title docs update docs: Deep links docs update May 27, 2026
Copy link
Copy Markdown
Member Author

charlesvien commented May 27, 2026

@charlesvien charlesvien marked this pull request as ready for review May 27, 2026 18:34
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 27, 2026

T-Rex T-Rex Logs

What T-Rex did

  • Ran a standalone runtime repro for the confetti helpers with window.matchMedia missing and confirmed the reduced-motion guard returns safely.
  • Tried to execute a focused confetti utility test in the local sandbox, but the environment could not run it due to missing dependencies and a test runner requiring a newer Node runtime.
  • Inspected the create-PR celebration flow and noted that the celebration is gated on successful PR creation and the persisted first-PR flag.
  • Inspected onboarding completion, Konami-code handling, and reset semantics, and did not observe a runtime repro in those paths.
Artifacts

Reduced-motion guard repro

  • Provides evidence of the standalone runtime repro for the reduced-motion guard in confetti helpers when window.matchMedia is absent.

Focused test startup log

  • Documents the startup outcome when attempting the focused confetti test, including dependency gaps and the need for a newer Node runtime.

T-Rex Ran code and verified through T-Rex

Prompt To Fix All With AI
Fix the following 1 code review issue. Work through them one at a time, proposing concise fixes.

---

### Issue 1 of 1
apps/code/src/renderer/features/settings/components/sections/AdvancedSettings.tsx:21-32
**Keep tour reset separate**

This removes the only standalone way to reset product tours. Previously a user could rerun tours without touching onboarding, but this action now also calls `resetOnboarding()` and `resetSetup()`. A user trying to replay tours will be forced through onboarding again and will lose setup/discovery state. Please keep a separate product-tour reset action or add a second button that only calls `resetTours()`.

Reviews (1): Last reviewed commit: "docs update" | Re-trigger Greptile

Copy link
Copy Markdown
Contributor

@adboio adboio left a comment

Choose a reason for hiding this comment

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

STANDARD DOCS UPDATE. LOOKS GOOD BOSS. 👍

Copy link
Copy Markdown
Member Author

charlesvien commented May 28, 2026

Merge activity

  • May 28, 1:10 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • May 28, 1:11 AM UTC: @charlesvien merged this pull request with Graphite.

@charlesvien charlesvien merged commit 71edb81 into main May 28, 2026
15 checks passed
@charlesvien charlesvien deleted the 05-27-docs_update branch May 28, 2026 01:11
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