Skip to content

feat(replays): Give users an informative message when loading Replay Preview fails#44000

Merged
ryan953 merged 3 commits intomasterfrom
ryan953/nice-replay-preview-load-fail
Mar 21, 2023
Merged

feat(replays): Give users an informative message when loading Replay Preview fails#44000
ryan953 merged 3 commits intomasterfrom
ryan953/nice-replay-preview-load-fail

Conversation

@ryan953
Copy link
Member

@ryan953 ryan953 commented Feb 2, 2023

If the replay fails to load for some reason we can show some reasons why, in future we should expand on this by linking out the product docs.

Before After
Issue Details - error issues missing replay - before issues misisng replay - after
Issue Details - preview preview - before preview - after

Fixes #43965
Fixes #45841
Fixes https://github.com/getsentry/team-replay/issues/47

@ryan953 ryan953 requested a review from a team February 2, 2023 00:30
@ryan953 ryan953 requested a review from a team as a code owner February 2, 2023 00:30
@github-actions github-actions bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Feb 2, 2023
@github-actions
Copy link
Contributor

github-actions bot commented Feb 2, 2023

size-limit report 📦

Path Size
src/sentry/static/sentry/dist/entrypoints/app.js 19.55 KB (+0.02% 🔺)
src/sentry/static/sentry/dist/entrypoints/sentry.css 32.99 KB (0%)

@github-actions
Copy link
Contributor

This pull request has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

@JoshFerge
Copy link
Member

do we still want to merge this?

@ryan953 ryan953 requested review from a team, billyvg and evanpurkhiser and removed request for billyvg and evanpurkhiser March 20, 2023 20:04
@ryan953 ryan953 force-pushed the ryan953/nice-replay-preview-load-fail branch from 5633515 to 863f681 Compare March 20, 2023 20:06
@ryan953 ryan953 requested a review from a team as a code owner March 20, 2023 20:06
Co-authored-by: Billy Vong <billyvg@users.noreply.github.com>
@ryan953 ryan953 enabled auto-merge (squash) March 21, 2023 00:29
@ryan953 ryan953 merged commit aca63c7 into master Mar 21, 2023
@ryan953 ryan953 deleted the ryan953/nice-replay-preview-load-fail branch March 21, 2023 00:32
@github-actions github-actions bot locked and limited conversation to collaborators Apr 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

4 participants