Skip to content

feat(seer-infra-telemetry): Add Datadog PAT connect flow to monitoring providers UI#118382

Merged
shashjar merged 1 commit into
masterfrom
shashjar/add-datadog-pat-connect-flow-to-monitoring-providers-UI
Jun 24, 2026
Merged

feat(seer-infra-telemetry): Add Datadog PAT connect flow to monitoring providers UI#118382
shashjar merged 1 commit into
masterfrom
shashjar/add-datadog-pat-connect-flow-to-monitoring-providers-UI

Conversation

@shashjar

@shashjar shashjar commented Jun 24, 2026

Copy link
Copy Markdown
Member

Resolves CW-1542.

See the Seer Advanced Settings page for more.

Adds a modal to the monitoring providers UI for connecting Datadog via Personal Access Token (PAT), as an alternative to OAuth. The PAT modal collects the access token (password-masked) and the Datadog site (via a selection from the list of valid/possible sites). On connection creation, it POSTs {access_token, site} to the existing backend endpoint, which returns 204. The disconnect flow works identically for both OAuth providers and PAT providers.

Gated behind the organization seer-infra-telemetry feature flag.

Main page

image

After attempting to connect with an invalid token or incorrect site

image

Main page after successful provider connection

image

Confirmation modal for disconnecting a provider

image

@linear-code

linear-code Bot commented Jun 24, 2026

Copy link
Copy Markdown

CW-1542

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jun 24, 2026
@shashjar shashjar requested review from srest2021 and wedamija June 24, 2026 19:14
@shashjar shashjar marked this pull request as ready for review June 24, 2026 19:14
@shashjar shashjar requested a review from a team as a code owner June 24, 2026 19:14

@srest2021 srest2021 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should we feature flag this?

@shashjar

Copy link
Copy Markdown
Member Author

@srest2021 the monitoring providers section is already feature flagged (so we won't display that section on Seer Advanced Settings without the flag) -

{organization.features.includes('seer-infra-telemetry') && (
<MonitoringProvidersSection />
)}

@srest2021 srest2021 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Lgtm for internal testing

@shashjar shashjar merged commit dc75f68 into master Jun 24, 2026
73 checks passed
@shashjar shashjar deleted the shashjar/add-datadog-pat-connect-flow-to-monitoring-providers-UI branch June 24, 2026 20:23
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