feat(seer): Add a reminder for Seer orgs who have not completed onboarding by ryan953 · Pull Request #106931 · getsentry/sentry

ryan953 · 2026-01-24T01:41:01Z

Adds a little 'reminder' widget to the sidebar, replacing the Prevent icon. The goal is to nudge people who have subscribed to Seer, and get them to configure it fully.

We only show the reminder if the org has not fully completed setup, using the same conditions that we use in the wizard (useSeerOnboardingStep).

Org has Flag	Img
`code-review-beta`
`seer-added`
`seat-based-seer-enabled`
mobile view

If they're using legacy Seer still, then the page they land on will correspond to which data is missing

Settings	Repos

Fixes https://linear.app/getsentry/issue/CW-679/help-existing-seer-orgs-onboard-into-code-review-before-updating

…fully

linear · 2026-01-24T01:41:07Z

CW-679 Help existing Seer orgs onboard into Code Review before updating subscription

static/gsApp/views/seerAutomation/seerAutomation.tsx

ryan953 · 2026-01-26T19:42:54Z

static/gsApp/views/seerAutomation/components/seerAutomationProjectList.tsx

+            <SimpleTable.Empty>
              {t('No projects found matching "%(search)s"', {search})}
-            </div>
+            </SimpleTable.Empty>


Before this had no padding, text was on the left.

After it looks like this now:

…inder

…iew beta. we show the tip if theyre not done onboarding

ryan953 · 2026-01-27T00:29:06Z

static/app/views/nav/primary/whatsNew/whatsNew.tsx


-  @media (min-height: 800px) {
+  /* TODO(ryan953): Make this shorter once showPreventNav() is removed from PrimaryNavigationItems */
+  @media (min-height: 724px) {


Tall Borderline Small (icons hidden)

ryan953 · 2026-01-27T00:39:51Z

static/app/router/routes.tsx

-      path: 'ai-code-review/',
-      children: [
-        {
-          component: make(() => import('sentry/views/prevent/preventAI/wrapper')),
-          children: [
-            {
-              index: true,
-              redirectTo: 'new/',
-            },
-            {
-              path: 'new/',
-              component: make(() => import('sentry/views/prevent/preventAI/onboarding')),
-            },
-          ],
-        },
-      ],
-    },


removed these

inlined the rest, the codecovChildren pattern is not needed.

ryan953 · 2026-01-27T00:40:49Z

static/app/views/prevent/pathnames.ts

used in one spot, inlined to that.

ryan953 · 2026-01-27T00:41:04Z

static/app/views/prevent/settings.tsx

used in a couple spots only. inlined to there.

static/app/views/nav/primary/index.tsx

static/app/router/routes.tsx

static/app/views/nav/primary/index.tsx

static/app/router/routes.tsx

static/gsApp/components/primaryNavSeerConfigReminder.tsx

static/app/components/commandPalette/useGlobalCommandPaletteActions.tsx

static/gsApp/views/seerAutomation/components/seerAutomationProjectList.tsx

billyvg

Is there no way to dismiss the reminder?

static/gsApp/components/primaryNavSeerConfigReminder.tsx

billyvg · 2026-01-28T18:40:05Z

static/gsApp/components/primaryNavSeerConfigReminder.tsx

+}
+
+const SeerButton = styled(SidebarButton)`
+  display: none;


Why this is hidden?

I pulled this CSS from static/app/views/nav/primary/whatsNew/whatsNew.tsx.

It's how the sidebar works i guess, idk.
I adjusted the @media so that things stay visible for longer (stays visible when the window is shorter than before, we have fewer main nav items so there's room).

I think it'll get refactored when design eng takes ownership of the nav, which is in design stages afaik

static/gsApp/views/seerAutomation/components/seerAutomationProjectList.tsx

JonasBa · 2026-01-28T18:55:09Z

@ryan this is just visual feedback, but could we push the button to the right side of the popover and increase the gap between the text and the button? I think that would help the flow a bit better + naturally fit with the spacing of the text

ryan953 · 2026-01-28T19:01:46Z

this is just visual feedback, but could we push the button to the right side of the popover and increase the gap between the text and the button? I think that would help the flow a bit better + naturally fit with the spacing of the text

@JonasBa done!

it's also hard to adjust the width of this thing, if i could make it narrower i would. something to follow up on

cursor

Cursor Bugbot has reviewed your changes and found 2 potential issues.

^{Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.}

cursor · 2026-01-28T19:27:27Z

static/gsApp/views/seerAutomation/seerAutomation.tsx

+          {tab === 'settings' ? (
+            <Fragment>
+              <SeerAutomationProjectList />
+              <br />


Using <br /> for spacing between components

Low Severity

A <br /> element is used to add spacing between SeerAutomationProjectList and SeerAutomationDefault. This is an anti-pattern in a component-based codebase that uses a layout system with proper spacing utilities. The surrounding Stack component with gap="lg" should handle this, or a wrapper with margin/padding should be used instead.

cursor · 2026-01-28T19:27:27Z

static/gsApp/components/primaryNavSeerConfigReminder.tsx

+  @media (min-height: 724px) {
+    display: flex;
+  }
+`;


Duplicated media query breakpoint with same TODO

Low Severity

The SeerButton styled component duplicates the exact same pattern from WhatsNewButton - including the magic number 724px, the identical TODO comment about showPreventNav(), and the display: none/display: flex toggle. This breakpoint value and associated styling pattern should be extracted into a shared constant or utility to avoid maintenance burden and ensure consistent updates.

Additional Locations (1)

static/app/views/nav/primary/whatsNew/whatsNew.tsx#L156-L164

JonasBa · 2026-01-28T19:44:12Z

this is just visual feedback, but could we push the button to the right side of the popover and increase the gap between the text and the button? I think that would help the flow a bit better + naturally fit with the spacing of the text

@JonasBa done!

it's also hard to adjust the width of this thing, if i could make it narrower i would. something to follow up on

Perfect! A narrower layout would be ideal here, ideally the width of the title or just a tad more would probably work well

…rding (#106931) Adds a little 'reminder' widget to the sidebar, replacing the Prevent icon. The goal is to nudge people who have subscribed to Seer, and get them to configure it fully. We only show the reminder if the org has not fully completed setup, using the same conditions that we use in the wizard (`useSeerOnboardingStep`). | Org has Flag | Img | | --- | --- | | `code-review-beta` | <img width="468" height="182" alt="SCR-20260126-oamm" src="https://github.com/user-attachments/assets/b28d95ba-e438-4d37-94a3-b33e8ef46e80" /> | `seer-added` | <img width="473" height="185" alt="SCR-20260126-oaps" src="https://github.com/user-attachments/assets/17295abe-90c0-43c0-9dbc-79a9747b1c5c" /> | `seat-based-seer-enabled` | <img width="469" height="194" alt="SCR-20260126-oatv" src="https://github.com/user-attachments/assets/ec0009b8-dd25-4b2e-af54-2281b14fe276" /> | mobile view | <img width="442" height="785" alt="SCR-20260126-obqo" src="https://github.com/user-attachments/assets/f965fff2-3a21-44fe-834a-84bc1c22a015" /> If they're using legacy Seer still, then the page they land on will correspond to which data is missing | Settings | Repos | | --- | --- | | <img width="1320" height="950" alt="SCR-20260126-olcg" src="https://github.com/user-attachments/assets/eacae6a2-ceac-4b96-8cc1-129f7276fbed" /> | <img width="1320" height="950" alt="SCR-20260126-okzc" src="https://github.com/user-attachments/assets/f3ec662b-bf80-47cd-be61-a3c3602ce779" /> Fixes https://linear.app/getsentry/issue/CW-679/help-existing-seer-orgs-onboard-into-code-review-before-updating

…tegration (#107288) For other integrations, like BitBucket and GitLab, we dont want to send a reminder because these things should be as setup as possible already. The extra steps are github only right now (but coming soon to more SCMs!) Followup of: #106931 Fixes CW-720

ryan953 added 3 commits January 23, 2026 17:02

feat(seer): Add a nav item to nudge existing seer users to set it up …

412ab41

…fully

feat(seer): Add the repos tab to the legacy seer settings

7402399

wip: banner to get people to install github integration

ae6bb0b

ryan953 requested review from a team as code owners January 24, 2026 01:41

github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jan 24, 2026

vercel bot deployed to Preview January 24, 2026 01:43 View deployment

cursor bot reviewed Jan 24, 2026

View reviewed changes

static/gsApp/views/seerAutomation/seerAutomation.tsx Outdated Show resolved Hide resolved

ryan953 marked this pull request as draft January 24, 2026 01:51

fix swapped tabs

afc5f87

vercel bot deployed to Preview January 26, 2026 19:04 View deployment

ryan953 added 2 commits January 26, 2026 11:06

direct link to the new repo tab

4707bea

improve empty message styles

6d17d55

ryan953 commented Jan 26, 2026

View reviewed changes

vercel bot deployed to Preview January 26, 2026 19:44 View deployment

ryan953 added 2 commits January 26, 2026 14:47

remove /prevent/ai-code-review/new/ page

3627339

cleanup remaining prevent tests&tokens pages and settings

e537ff1

ryan953 force-pushed the ryan953/CW-679-seer-migration-onboarding branch from c5d07a6 to e537ff1 Compare January 26, 2026 23:14

vercel bot deployed to Preview January 26, 2026 23:17 View deployment

ryan953 added 2 commits January 26, 2026 15:35

Adjust permissions, and css conditons for showing the seer config rem…

1189c7e

…inder

update copy for three groups: new seer, legacy autofix only, code-rev…

98761e5

…iew beta. we show the tip if theyre not done onboarding

vercel bot deployed to Preview January 27, 2026 00:12 View deployment

ryan953 commented Jan 27, 2026

View reviewed changes

static/app/views/prevent/pathnames.ts Outdated

Copy link
Copy Markdown

Member Author

ryan953 Jan 27, 2026

Choose a reason for hiding this comment

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

used in one spot, inlined to that.

ryan953 commented Jan 27, 2026

View reviewed changes

static/app/views/prevent/settings.tsx Outdated

Copy link
Copy Markdown

Member Author

ryan953 Jan 27, 2026

Choose a reason for hiding this comment

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

used in a couple spots only. inlined to there.

fix tests

1f338f0

ryan953 marked this pull request as ready for review January 27, 2026 00:45

ryan953 requested a review from a team as a code owner January 27, 2026 00:45

sentry bot reviewed Jan 27, 2026

View reviewed changes

static/app/views/nav/primary/index.tsx Outdated Show resolved Hide resolved

static/app/router/routes.tsx Show resolved Hide resolved

vercel bot deployed to Preview January 27, 2026 00:48 View deployment

cursor bot reviewed Jan 27, 2026

View reviewed changes

static/app/views/nav/primary/index.tsx Outdated Show resolved Hide resolved

static/app/router/routes.tsx Outdated Show resolved Hide resolved

Update copy

c98f7d4

vercel bot deployed to Preview January 27, 2026 18:49 View deployment

Update redirects and command palette

477cab4

sentry bot reviewed Jan 27, 2026

View reviewed changes

static/gsApp/components/primaryNavSeerConfigReminder.tsx Show resolved Hide resolved

vercel bot deployed to Preview January 27, 2026 18:53 View deployment

cursor bot reviewed Jan 27, 2026

View reviewed changes

static/gsApp/components/primaryNavSeerConfigReminder.tsx Show resolved Hide resolved

static/gsApp/components/primaryNavSeerConfigReminder.tsx Show resolved Hide resolved

static/app/components/commandPalette/useGlobalCommandPaletteActions.tsx Show resolved Hide resolved

ryan953 changed the title ~~WIP - seer settings for migrating orgs~~ feat(seer): Add a reminder for Seer orgs who have not completed onboarding Jan 27, 2026

fix logic in command palette, and keep CTA link logic matching the copy

fe10ec4

vercel bot deployed to Preview January 27, 2026 19:45 View deployment

cursor bot reviewed Jan 27, 2026

View reviewed changes

static/gsApp/views/seerAutomation/components/seerAutomationProjectList.tsx Show resolved Hide resolved

billyvg approved these changes Jan 28, 2026

View reviewed changes

ryan953 closed this Jan 28, 2026

ryan953 reopened this Jan 28, 2026

ryan953 added 2 commits January 28, 2026 11:02

adjust popup spacing/alignment

3b7c533

remove todo, not needed

d28360f

vercel bot deployed to Preview January 28, 2026 19:17 View deployment

cursor bot reviewed Jan 28, 2026

View reviewed changes

ryan953 merged commit 8dca707 into master Jan 28, 2026
53 checks passed

ryan953 deleted the ryan953/CW-679-seer-migration-onboarding branch January 28, 2026 20:48

ryan953 mentioned this pull request Jan 30, 2026

feat(seer): Only remind people to setup Seer if the org has GitHub integration #107288

Merged

github-actions bot locked and limited conversation to collaborators Feb 13, 2026

Uh oh!

Conversation

ryan953 commented Jan 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

linear bot commented Jan 24, 2026

Uh oh!

Uh oh!

ryan953 Jan 26, 2026

Choose a reason for hiding this comment

Uh oh!

ryan953 Jan 27, 2026

Choose a reason for hiding this comment

Uh oh!

ryan953 Jan 27, 2026

Choose a reason for hiding this comment

Uh oh!

ryan953 Jan 27, 2026

Choose a reason for hiding this comment

Uh oh!

ryan953 Jan 27, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

billyvg left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

billyvg Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

ryan953 Jan 28, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

JonasBa commented Jan 28, 2026

Uh oh!

ryan953 commented Jan 28, 2026

Uh oh!

cursor bot left a comment

Choose a reason for hiding this comment

Uh oh!

cursor bot Jan 28, 2026

Choose a reason for hiding this comment

Using <br /> for spacing between components

Uh oh!

cursor bot Jan 28, 2026

Choose a reason for hiding this comment

Duplicated media query breakpoint with same TODO

Uh oh!

JonasBa commented Jan 28, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

ryan953 commented Jan 24, 2026 •

edited

Loading

Using `<br />` for spacing between components