Skip to content

ref(seer): Combine AutofixRepositories components into one#117618

Closed
ryan953 wants to merge 0 commit into
masterfrom
ryan953/ref-combine-AutofixRepositories
Closed

ref(seer): Combine AutofixRepositories components into one#117618
ryan953 wants to merge 0 commit into
masterfrom
ryan953/ref-combine-AutofixRepositories

Conversation

@ryan953

@ryan953 ryan953 commented Jun 14, 2026

Copy link
Copy Markdown
Member

This meant adding a includeInstructions prop to the new version, which revealed that the inputs needed to be debounced. apparently saving a default-branch name before was problematic

With instructions added
SCR-20260614-lqlb

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jun 14, 2026
@ryan953 ryan953 changed the base branch from master to ryan953/extract-pick-project June 14, 2026 21:05
@ryan953 ryan953 changed the base branch from ryan953/extract-pick-project to ryan953/autofix-repo-stories June 14, 2026 21:07
@github-actions

github-actions Bot commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

@ryan953 ryan953 marked this pull request as ready for review June 15, 2026 06:00
@ryan953 ryan953 requested review from a team as code owners June 15, 2026 06:00
Comment thread static/app/components/seer/projectDetails/autofixRepositoriesList.tsx Outdated
Comment thread static/app/components/seer/projectDetails/autofixRepositoriesItem.tsx Outdated
Comment thread static/app/components/seer/projectDetails/autofixRepositoriesList.tsx Outdated
Comment thread static/app/components/seer/legacy/autofixRepositories.tsx Outdated
Comment thread static/app/components/seer/projectDetails/autofixRepositoriesList.tsx Outdated
Comment thread static/app/components/seer/projectDetails/autofixRepositoriesItem.tsx Outdated
@ryan953 ryan953 force-pushed the ryan953/ref-combine-AutofixRepositories branch from 3894f91 to a76ecf7 Compare June 15, 2026 15:50
@ryan953 ryan953 force-pushed the ryan953/autofix-repo-stories branch from 86bd844 to 6687049 Compare June 15, 2026 15:50

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit a76ecf7. Configure here.

Comment thread static/app/components/seer/legacy/autofixRepositories.tsx Outdated
Comment thread static/app/components/seer/legacy/autofixRepositories.tsx Outdated
Base automatically changed from ryan953/autofix-repo-stories to master June 15, 2026 16:02
@ryan953 ryan953 requested review from a team as code owners June 15, 2026 16:02
@ryan953 ryan953 closed this Jun 15, 2026
@ryan953 ryan953 force-pushed the ryan953/ref-combine-AutofixRepositories branch from a76ecf7 to 753461c Compare June 15, 2026 20:34
@ryan953 ryan953 deleted the ryan953/ref-combine-AutofixRepositories branch June 15, 2026 20:36
@ryan953 ryan953 restored the ryan953/ref-combine-AutofixRepositories branch June 15, 2026 20:37
ryan953 added a commit that referenced this pull request Jun 16, 2026
…points (#117740)

This meant adding a `includeInstructions` prop to the new version, which
revealed that the inputs needed to be debounced. apparently saving a
default-branch name before was problematic

With instructions added
<img width="819" height="547" alt="SCR-20260614-lqlb"
src="https://github.com/user-attachments/assets/325af416-9b42-4de1-b190-f8bc874ecd79"
/>


Note: these were originally stacked into
#117618 and
#117630 but i messed that up
somehow to update things on master. So now they're together and ready to
go. No changes made since before.

---------

Co-authored-by: Claude Opus 4 <noreply@anthropic.com>
Co-authored-by: Ryan Albrecht <ryan953@MLHJDQR9VJ.local>
Co-authored-by: billy <billy@sentry.io>
Co-authored-by: sentry-junior[bot] <264270552+sentry-junior[bot]@users.noreply.github.com>
Co-authored-by: getsantry[bot] <66042841+getsantry[bot]@users.noreply.github.com>
sehr-m pushed a commit that referenced this pull request Jun 23, 2026
…points (#117740)

This meant adding a `includeInstructions` prop to the new version, which
revealed that the inputs needed to be debounced. apparently saving a
default-branch name before was problematic

With instructions added
<img width="819" height="547" alt="SCR-20260614-lqlb"
src="https://github.com/user-attachments/assets/325af416-9b42-4de1-b190-f8bc874ecd79"
/>


Note: these were originally stacked into
#117618 and
#117630 but i messed that up
somehow to update things on master. So now they're together and ready to
go. No changes made since before.

---------

Co-authored-by: Claude Opus 4 <noreply@anthropic.com>
Co-authored-by: Ryan Albrecht <ryan953@MLHJDQR9VJ.local>
Co-authored-by: billy <billy@sentry.io>
Co-authored-by: sentry-junior[bot] <264270552+sentry-junior[bot]@users.noreply.github.com>
Co-authored-by: getsantry[bot] <66042841+getsantry[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant