Skip to content

fix(aci): Hide 'All environments' option in uptime detector form#106578

Merged
malwilley merged 1 commit intomasterfrom
malwilley/fix/hide-all-environments-option-in-uptime-detector-form
Jan 20, 2026
Merged

fix(aci): Hide 'All environments' option in uptime detector form#106578
malwilley merged 1 commit intomasterfrom
malwilley/fix/hide-all-environments-option-in-uptime-detector-form

Conversation

@malwilley
Copy link
Member

@malwilley malwilley commented Jan 20, 2026

Fixes ID-1235

Also does a small refactor of the props to allow better configuration of the environment field

CleanShot 2026-01-20 at 09 50 52@2x

@malwilley malwilley requested a review from a team January 20, 2026 17:51
@malwilley malwilley requested a review from a team as a code owner January 20, 2026 17:51
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jan 20, 2026
@linear
Copy link

linear bot commented Jan 20, 2026

@malwilley malwilley enabled auto-merge (squash) January 20, 2026 17:54
Copy link
Contributor

@cursor cursor bot left a comment

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.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

import {UptimeDetectorVerificationSection} from 'sentry/views/detectors/components/forms/uptime/verification';

const ENVIRONMENT_CONFIG: React.ComponentProps<typeof DetectorBaseFields>['environment'] =
{includeAllEnvironments: false, fieldProps: {required: true}};
Copy link
Contributor

Choose a reason for hiding this comment

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

Empty required dropdown when project has no environments

Low Severity

When a project has no environments configured, the uptime detector form now shows an empty environment dropdown that is marked as required. Users cannot complete the form because there are no options to select. Previously, "All Environments" served as a fallback option. The combination of includeAllEnvironments: false with required: true creates an impossible-to-complete form state when environments is empty.

Additional Locations (1)

Fix in Cursor Fix in Web

@malwilley malwilley merged commit e5de50b into master Jan 20, 2026
55 checks passed
@malwilley malwilley deleted the malwilley/fix/hide-all-environments-option-in-uptime-detector-form branch January 20, 2026 18:00
@github-actions github-actions bot locked and limited conversation to collaborators Feb 5, 2026
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.

2 participants