Skip to content

feat(replays): Add a note about cross-origin (CDN) Replay Network request status codes#48640

Merged
ryan953 merged 5 commits into
masterfrom
ryan953/replay-network-cached-assets
May 9, 2023
Merged

feat(replays): Add a note about cross-origin (CDN) Replay Network request status codes#48640
ryan953 merged 5 commits into
masterfrom
ryan953/replay-network-cached-assets

Conversation

@ryan953

@ryan953 ryan953 commented May 5, 2023

Copy link
Copy Markdown
Member

Lots of 0 in the status column now, but the tooltip provides some new tips for how to improve things.

Also, blank cells (in all columns) show -- now instead of being blank. There are not that many blank cells overall.

SCR-20230505-mayq

For Reference: https://developer.mozilla.org/en-US/docs/Web/API/PerformanceResourceTiming/responseStatus#cross-origin_response_status_codes

Relates to https://github.com/getsentry/team-replay/issues/35

Other copy changes, and switching from <kbd> to <code> looks like this:

Title Screen Shot
Before (black highlights) screenshot_2023-05-05_at_4 01 49_pm
After (monospace font) SCR-20230505-mcqu

@ryan953 ryan953 requested a review from a team as a code owner May 5, 2023 17:53
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label May 5, 2023
Comment thread static/app/views/replays/detail/network/networkTableCell.tsx
Comment thread static/app/views/replays/detail/network/networkHeaderCell.tsx Outdated
Comment thread static/app/views/replays/detail/network/networkHeaderCell.tsx Outdated
@ryan953 ryan953 requested review from JonasBa and billyvg May 5, 2023 20:33
@ryan953

ryan953 commented May 5, 2023

Copy link
Copy Markdown
Member Author

Added a bunch of other copy changes to make things more consistent in this area.
ie: "Payloads" -> "Bodies"

Comment thread static/app/views/replays/detail/network/details/onboarding.tsx
@ryan953 ryan953 force-pushed the ryan953/replay-network-cached-assets branch from 9ff3586 to f4be952 Compare May 8, 2023 21:32
@ryan953 ryan953 merged commit d241906 into master May 9, 2023
@ryan953 ryan953 deleted the ryan953/replay-network-cached-assets branch May 9, 2023 00:16
@github-actions github-actions Bot locked and limited conversation to collaborators May 24, 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

Development

Successfully merging this pull request may close these issues.

4 participants