Skip to content

ref(profiling): rename explore/profiling URL to explore/profiles#115627

Merged
JoshuaKGoldberg merged 2 commits into
masterfrom
joshuakgoldberg/ref/rename-explore-profiling-to-profiles
May 26, 2026
Merged

ref(profiling): rename explore/profiling URL to explore/profiles#115627
JoshuaKGoldberg merged 2 commits into
masterfrom
joshuakgoldberg/ref/rename-explore-profiling-to-profiles

Conversation

@JoshuaKGoldberg
Copy link
Copy Markdown
Member

@JoshuaKGoldberg JoshuaKGoldberg commented May 15, 2026

Switches:

  • Route path: explore/profiling/ to explore/profiles/ with a <Redirect>
  • Page title: Profiling to Profiles
Before After
image image

Note that user-facing docs like https://docs.sentry.io/product/explore/profiling still say "Profiling".

Starts on EXP-955.

Unifies the naming in Explore so the Profiles section matches the
pattern of other items (Traces, Logs, Metrics). Previously the URL
and page title used 'Profiling' while the sidebar nav used 'Profiles'.

Changes:
- Route path: explore/profiling/ → explore/profiles/
- Document title and page heading: 'Profiling' → 'Profiles'
- Breadcrumb label: 'Profiling' → 'Profiles'
- Sidebar nav link: updated to point at profiles/
- Adds explore/profiling/* → explore/profiles/* redirect to
  preserve bookmarks and old links with query params
- Legacy /profiling/ redirect now targets /explore/profiles/

Refs EXP-955
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 15, 2026

EXP-955

@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 15, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 15, 2026

📊 Type Coverage Diff

✅ No new type safety issues introduced. Coverage: 93.56%

@JoshuaKGoldberg JoshuaKGoldberg marked this pull request as ready for review May 15, 2026 14:07
@JoshuaKGoldberg JoshuaKGoldberg requested review from a team as code owners May 15, 2026 14:07
Copy link
Copy Markdown
Contributor

@nsdeschenes nsdeschenes left a comment

Choose a reason for hiding this comment

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

Looks all good to me, wondering if there's a test we can add in to ensure that the redirect happens, so we don't have a regression or anything 👀

@JoshuaKGoldberg JoshuaKGoldberg changed the title ref(profiling): Rename explore/profiling URL to explore/profiles ref(profiling): rename explore/profiling URL to explore/profiles May 18, 2026
@JoshuaKGoldberg
Copy link
Copy Markdown
Member Author

Good call, asking in Slack...

@JoshuaKGoldberg
Copy link
Copy Markdown
Member Author

JoshuaKGoldberg commented May 19, 2026

👍 Set up in:

  1. feat(profiles): add backend redirect for /explore/profiling #115819: includes testing on the backend
  2. ref(profiling): remove now-unnecessary frontend redirects #115821: removes the frontend code

@JoshuaKGoldberg
Copy link
Copy Markdown
Member Author

Design engineering is still discussing, converting to draft for now.

@JoshuaKGoldberg JoshuaKGoldberg marked this pull request as draft May 20, 2026 13:16
@JoshuaKGoldberg
Copy link
Copy Markdown
Member Author

Resolved that we don't need the full backend redirects, that's older behavior. I'll get this merged up and back in review. 🚀

@JoshuaKGoldberg JoshuaKGoldberg marked this pull request as ready for review May 26, 2026 16:10
Copy link
Copy Markdown
Contributor

@nsdeschenes nsdeschenes left a comment

Choose a reason for hiding this comment

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

:shipit:

@JoshuaKGoldberg JoshuaKGoldberg merged commit c6327bd into master May 26, 2026
70 checks passed
@JoshuaKGoldberg JoshuaKGoldberg deleted the joshuakgoldberg/ref/rename-explore-profiling-to-profiles branch May 26, 2026 18:00
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