Skip to content

chore(anomaly detection): Remove unused code#89998

Merged
ceorourke merged 1 commit into
masterfrom
ceorourke/rm-anom-charts-fe
Apr 22, 2025
Merged

chore(anomaly detection): Remove unused code#89998
ceorourke merged 1 commit into
masterfrom
ceorourke/rm-anom-charts-fe

Conversation

@ceorourke

@ceorourke ceorourke commented Apr 21, 2025

Copy link
Copy Markdown
Member

An attempt at the FE counterpart to https://github.com/getsentry/sentry/pull/89406/files to remove code around a feature flag that we never ended up using.

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Apr 21, 2025
@ceorourke ceorourke marked this pull request as ready for review April 21, 2025 20:11
@ceorourke ceorourke requested review from a team as code owners April 21, 2025 20:11
@ceorourke

Copy link
Copy Markdown
Member Author

Took a while but I ran this locally to triple check the chart looks the same as it does today (set all the anomaly detection flags, ran Seer locally, and faked the historical data to create a dynamic alert). I'll merge this tomorrow morning.

@ceorourke ceorourke merged commit dc9bf43 into master Apr 22, 2025
@ceorourke ceorourke deleted the ceorourke/rm-anom-charts-fe branch April 22, 2025 17:28
billyvg pushed a commit that referenced this pull request Apr 22, 2025
An attempt at the FE counterpart to
https://github.com/getsentry/sentry/pull/89406/files to remove code
around a feature flag that we never ended up using.
andrewshie-sentry pushed a commit that referenced this pull request Apr 22, 2025
An attempt at the FE counterpart to
https://github.com/getsentry/sentry/pull/89406/files to remove code
around a feature flag that we never ended up using.
@github-actions github-actions Bot locked and limited conversation to collaborators May 8, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants