Skip to content

chore(ACI): Remove legacy rule creation from project onboarding - #121971

Open
ceorourke wants to merge 3 commits into
masterfrom
ceorourke/rm-onboarding-rule-creation
Open

chore(ACI): Remove legacy rule creation from project onboarding#121971
ceorourke wants to merge 3 commits into
masterfrom
ceorourke/rm-onboarding-rule-creation

Conversation

@ceorourke

@ceorourke ceorourke commented Aug 13, 2026

Copy link
Copy Markdown
Member

Remove legacy rule creation from the onboarding checklist - the endpoint it calls is deprecated and scheduled to be deleted next week. A default workflow is created with a signal on project creation, but we may want to replace this with an onboarding item to create a custom workflow in the future.

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Aug 13, 2026
let ruleId: string | undefined;
if (alertRuleConfig.shouldCreateCustomRule) {
const ruleData = await api.requestPromise(
`/projects/${organization.slug}/${projectData.slug}/rules/`,

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This endpoint is deprecated and will be deleted soon

@ceorourke
ceorourke marked this pull request as ready for review August 13, 2026 17:10
@ceorourke
ceorourke requested review from a team as code owners August 13, 2026 17:10
@ceorourke

Copy link
Copy Markdown
Member Author

@getsentry/value-discovery team I noticed you're working on some related stuff so I wanted to make sure you were aware of this - it needs to be removed but if it makes sense to replace it with a workflow, reach out to the issue-workflow team for help if needed.

@ceorourke
ceorourke requested review from a team and removed request for a team August 13, 2026 17:12

@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 18247bf. Configure here.

Comment thread static/app/components/onboarding/useCreateProjectAndRules.ts
Comment thread static/app/components/onboarding/useCreateProjectAndRules.ts
@ceorourke
ceorourke force-pushed the ceorourke/rm-onboarding-rule-creation branch from 1db2d75 to a11f7ac Compare August 13, 2026 20:09
@ceorourke
ceorourke marked this pull request as ready for review August 13, 2026 20:13
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