Skip to content

fix(profiles): indicate invalid page URL state as error#115897

Merged
JoshuaKGoldberg merged 4 commits into
masterfrom
profiles-invalid-page-visualize
May 21, 2026
Merged

fix(profiles): indicate invalid page URL state as error#115897
JoshuaKGoldberg merged 4 commits into
masterfrom
profiles-invalid-page-visualize

Conversation

@JoshuaKGoldberg
Copy link
Copy Markdown
Member

@JoshuaKGoldberg JoshuaKGoldberg commented May 20, 2026

Uses our snazzy <LoadingError> Scraps component to make it clear that the page is invalid.

Before After
image image

Closes PRO-43.

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

github-actions Bot commented May 20, 2026

📊 Type Coverage Diff

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

@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 20, 2026

PRO-43

@JoshuaKGoldberg JoshuaKGoldberg marked this pull request as ready for review May 20, 2026 19:02
@JoshuaKGoldberg JoshuaKGoldberg requested a review from a team as a code owner May 20, 2026 19:02
Comment thread static/app/views/explore/profiling/profilesProvider.tsx Outdated
Comment thread static/app/views/explore/profiling/profilesProvider.tsx
Comment thread static/app/views/explore/profiling/profilesProvider.tsx Outdated
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.

There are 2 total unresolved issues (including 1 from previous review).

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 38b32d8. Configure here.

Comment thread static/app/views/explore/profiling/profilesProvider.tsx
Comment thread static/app/views/explore/profiling/profilesProvider.tsx Outdated
Copy link
Copy Markdown
Contributor

@adrianviquez adrianviquez left a comment

Choose a reason for hiding this comment

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

Lgtm! The t() suggestion makes sense to address, otherwise approve

@JoshuaKGoldberg JoshuaKGoldberg merged commit d01dcbd into master May 21, 2026
71 checks passed
@JoshuaKGoldberg JoshuaKGoldberg deleted the profiles-invalid-page-visualize branch May 21, 2026 14:35
JonasBa pushed a commit that referenced this pull request May 21, 2026
Uses our snazzy [`<LoadingError>` Scraps
component](https://lucidsoftware.sentry.io/stories/product/components/loadingerror)
to make it clear that the page is invalid.

<table>
<thead>
<tr>
<th>Before</th>
<th>After</th>
</tr>
</thead>
<tbody>
<tr>
<td>
<img width="953" height="530" alt="image"
src="https://github.com/user-attachments/assets/8a0c46b3-51b3-43e0-9157-6a8bfc2acc2d"
/>
</td>
<td>
<img width="953" height="530" alt="image"
src="https://github.com/user-attachments/assets/3167b1cb-e8f4-425b-ae9c-9f7ef433e9bd"
/>
</td>
</tr>
</tbody>
</table>

Closes PRO-43.
natemoo-re pushed a commit that referenced this pull request May 21, 2026
Uses our snazzy [`<LoadingError>` Scraps
component](https://lucidsoftware.sentry.io/stories/product/components/loadingerror)
to make it clear that the page is invalid.

<table>
<thead>
<tr>
<th>Before</th>
<th>After</th>
</tr>
</thead>
<tbody>
<tr>
<td>
<img width="953" height="530" alt="image"
src="https://github.com/user-attachments/assets/8a0c46b3-51b3-43e0-9157-6a8bfc2acc2d"
/>
</td>
<td>
<img width="953" height="530" alt="image"
src="https://github.com/user-attachments/assets/3167b1cb-e8f4-425b-ae9c-9f7ef433e9bd"
/>
</td>
</tr>
</tbody>
</table>

Closes PRO-43.
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