Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

remove long-unused UI routes#63670

Merged
sqs merged 1 commit into
mainfrom
sqs/rm-old-ui-routes
Jul 5, 2024
Merged

remove long-unused UI routes#63670
sqs merged 1 commit into
mainfrom
sqs/rm-old-ui-routes

Conversation

@sqs

@sqs sqs commented Jul 5, 2024

Copy link
Copy Markdown
Member
  • Threads: 2+ years removed, from old code discussions functionality
  • Snippets: never a top-level route
  • Subscriptions: ~2 years removed, from old Stripe license key payment code
  • Views: 2+ years removed, experimental feature
  • Own: not a top-level route anymore, never on dotcom
  • Registry: ~1.5 years removed, was the extension registry UI (API routes still available for backcompat)
  • /search/cody: removed from dotcom ~1 month ago
  • App: removed 6 months ago, experimental and not used anymore

Test plan

Test that none of these routes are active on s2 or Sourcegraph.com.

- Threads: 2+ years removed, from old code discussions functionality
- Snippets: never a top-level route
- Subscriptions: ~2 years removed, from old Stripe license key payment code
- Views: 2+ years removed, experimental feature
- Own: not a top-level route anymore, never on dotcom
- Registry: ~1.5 years removed, was the extension registry UI (API routes still available for backcompat)
- `/search/cody`: removed from dotcom ~1 month ago
- App: removed 6 months ago, experimental and not used anymore
@sqs sqs requested a review from a team July 5, 2024 13:41
@cla-bot cla-bot Bot added the cla-signed label Jul 5, 2024
@sqs sqs merged commit fc011a1 into main Jul 5, 2024
@sqs sqs deleted the sqs/rm-old-ui-routes branch July 5, 2024 14:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants