Skip to content

fix(repos): Fix vsts sync error#113136

Merged
wedamija merged 2 commits intomasterfrom
danf/fix-vsts-error
Apr 16, 2026
Merged

fix(repos): Fix vsts sync error#113136
wedamija merged 2 commits intomasterfrom
danf/fix-vsts-error

Conversation

@wedamija
Copy link
Copy Markdown
Member

The links field isn't always present. Use webUrl instead, and fall back to an empty url as needed

Fixes SENTRY-5NAH

@wedamija wedamija requested a review from a team April 15, 2026 23:21
@wedamija wedamija requested review from a team as code owners April 15, 2026 23:21
@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 15, 2026
Copy link
Copy Markdown
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.

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 c788c06. Configure here.

Comment thread src/sentry/integrations/vsts/integration.py Outdated
Comment thread src/sentry/integrations/vsts/integration.py Outdated
The links field isn't always present. Use `webUrl` instead, and fall back to an empty url as needed

Fixes SENTRY-5NAH
Comment thread src/sentry/integrations/vsts/integration.py
@wedamija wedamija merged commit bca0f50 into master Apr 16, 2026
56 checks passed
@wedamija wedamija deleted the danf/fix-vsts-error branch April 16, 2026 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants