Skip to content
Snippets Groups Projects

Fix externalURLs alert logic

Merged Warren Gifford requested to merge fix-external-urls-destructure into main

Created by: felixfbecker

Fixes https://github.com/sourcegraph/customer/issues/118

The issue was that the code assumed the array always had an element, but it doesn't need to have. I also noticed that we didn't check whether we actually support the code host. With this PR, we take the first code host from external links that we support, and show nothing if there is none.

cc @dadlerj

Merge request reports

Merged by avatar (Jun 23, 2025 7:19am UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading