Skip to content

feat(seer): Enable bulk editing seer automation #91807

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 8 commits into from
May 16, 2025

Conversation

roaga
Copy link
Member

@roaga roaga commented May 16, 2025

Screenshot 2025-05-16 at 10 10 08 AM Screenshot 2025-05-16 at 10 10 04 AM Screenshot 2025-05-16 at 10 09 59 AM

@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 16, 2025
@roaga roaga marked this pull request as ready for review May 16, 2025 18:59
@roaga roaga requested a review from scttcper May 16, 2025 19:07
Co-authored-by: Scott Cooper <scttcper@gmail.com>
roaga and others added 2 commits May 16, 2025 13:38
Co-authored-by: Scott Cooper <scttcper@gmail.com>
Co-authored-by: Scott Cooper <scttcper@gmail.com>
@@ -49,8 +57,10 @@ function ProjectSeerSetting({project, orgSlug}: {orgSlug: string; project: Proje

export function SeerAutomationProjectList() {
const organization = useOrganization();
const {projects, fetching, fetchError} = useProjects();
const api = useApi();
const {projects, fetching, fetchError, reloadProjects} = useProjects();
Copy link
Member

Choose a reason for hiding this comment

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

reloadProjects here won't do anything since we're using useDetailedProject to fetch additional project details. see https://github.com/getsentry/sentry/blob/master/static/app/utils/useDetailedProject.tsx#L18-L27

Copy link
Member Author

Choose a reason for hiding this comment

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

gotcha, switching to invalidating the detailed project queries if that works

Co-authored-by: Scott Cooper <scttcper@gmail.com>
@roaga roaga merged commit 59474fb into master May 16, 2025
41 checks passed
@roaga roaga deleted the seer/bulk-automation-editing branch May 16, 2025 21:11
andrewshie-sentry pushed a commit that referenced this pull request May 20, 2025
<img width="1369" alt="Screenshot 2025-05-16 at 10 10 08 AM"
src="https://github.com/user-attachments/assets/d5e14874-7ac7-4b10-a129-22a671b14a6d"
/>
<img width="1333" alt="Screenshot 2025-05-16 at 10 10 04 AM"
src="https://github.com/user-attachments/assets/3b3afb53-d0a9-4b9b-af93-78393b9495b8"
/>
<img width="1380" alt="Screenshot 2025-05-16 at 10 09 59 AM"
src="https://github.com/user-attachments/assets/b1bf57d3-a187-4e94-a6f7-8dfbf10f1f6c"
/>

---------

Co-authored-by: Scott Cooper <scttcper@gmail.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.

2 participants