-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
feat(aci): connect automations + monitors views to router #83137
base: master
Are you sure you want to change the base?
Conversation
Bundle ReportChanges will increase total bundle size by 33.05kB (0.11%) ⬆️. This is within the configured threshold ✅ Detailed changes
|
721cec1
to
e665d9c
Compare
7c9cd5c
to
c8f1cd0
Compare
c8f1cd0
to
027f823
Compare
when stacking PRs please do them as drafts so they don't notify everyone when you rebase your branch |
Sorry about that everyone! No idea what happened here. Thanks for the heads up @asottile-sentry. |
14cf073
to
b2b7f9b
Compare
21d3fee
to
aaaac02
Compare
🚨 Warning: This pull request contains Frontend and Backend changes! It's discouraged to make changes to Sentry's Frontend and Backend in a single pull request. The Frontend and Backend are not atomically deployed. If the changes are interdependent of each other, they must be separated into two pull requests and be made forward or backwards compatible, such that the Backend or Frontend can be safely deployed independently. Have questions? Please ask in the |
ad952f4
to
14300ea
Compare
❌ 1 Tests Failed:
View the top 1 failed tests by shortest run time
To view more test analytics, go to the Test Analytics Dashboard |
aa0c18a
to
e54c234
Compare
aaaac02
to
3cd7d0c
Compare
3cd7d0c
to
40888d4
Compare
531bf19
to
a15685a
Compare
ACI M6.2: View Scaffolding
Connects new ACI views (Automations and Monitors) to router. Uses a new pattern where the route definitions are exported from the relevant directory rather than managed in the monolithic file.
✅ Closes
ACI-65