Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat(docs): session replay (limited preview) #15042

Merged
merged 14 commits into from
Jan 25, 2024
Merged

feat(docs): session replay (limited preview) #15042

merged 14 commits into from
Jan 25, 2024

Conversation

ally-sassman
Copy link
Contributor

@ally-sassman ally-sassman commented Oct 31, 2023 •
edited
Loading

PR for transitioning the Session Replay LP doc to the public docs site.

Jira: NR-169850

Note to self: Before merging, check to make sure all screenshots are in dark mode.

@ally-sassman ally-sassman added content requests related to docs site content from_tw Identifies issues/PRs from Tech Docs writers labels Oct 31, 2023
@ally-sassman ally-sassman self-assigned this Oct 31, 2023
@github-actions GitHub Actions
Copy link

Hi @ally-sassman 👋

Thanks for your pull request! Your PR is in a queue, and a writer will take a look soon. We generally publish small edits within one business day, and larger edits within three days.

Gatsby Cloud will automatically generate a preview of your request, and will comment with a link when the preview is ready (usually 20 to 30 minutes).

@gatsby-cloud Gatsby Cloud
Copy link

gatsby-cloud bot commented Oct 31, 2023 •
edited
Loading

✅ docs-website-develop deploy preview ready

@ally-sassman
Copy link
Contributor Author

Closing until we're closer to GA.

@ally-sassman ally-sassman reopened this Jan 18, 2024
@ally-sassman ally-sassman changed the title feat(docs): session replay GA feat(docs): session replay (limited preview) Jan 18, 2024
@ally-sassman ally-sassman marked this pull request as ready for review January 24, 2024 18:52
@github-actions github-actions bot requested a review from rhetoric101 January 24, 2024 18:52
@ally-sassman ally-sassman requested review from bradleycamacho and removed request for rhetoric101 January 24, 2024 18:52
bradleycamacho
bradleycamacho previously approved these changes Jan 25, 2024
View reviewed changes
Copy link
Contributor

@bradleycamacho bradleycamacho left a comment

Choose a reason for hiding this comment

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

This looks really good! Just a few copyedits here and there. I think the structure looks great, good job!

The only other note I have is that you should explore cross-linking from other browser docs. You may not want to do it now as its still in preview, but I think the browser docs should be updated to mention and link to session replay in quite a few places.

src/content/docs/browser/browser-monitoring/browser-pro-features/session-replay.mdx Outdated Show resolved Hide resolved
src/content/docs/browser/browser-monitoring/browser-pro-features/session-replay.mdx

Here's an example of how a session replay looks:

<img
Copy link
Contributor

Choose a reason for hiding this comment

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

The screenshot is fine, but I'd actually try to put a video here. I actually tried to click on it assuming it was a video, and I think session replay is the perfect product to actually use a video for (or gif)

src/content/docs/browser/browser-monitoring/browser-pro-features/session-replay.mdx Show resolved Hide resolved
src/content/docs/browser/browser-monitoring/browser-pro-features/session-replay.mdx Show resolved Hide resolved
src/content/docs/browser/browser-monitoring/browser-pro-features/session-replay.mdx
* Strategic payload harvesting: It only sends a large (compressed) payload when the full DOM is captured, which typically only happens when the replay feature is first imported. After that, it will only send small payloads that relate to the end-user's actions.
</Collapser>

<Collapser
Copy link
Contributor

Choose a reason for hiding this comment

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

This info is already mentioned above. I'd cut this.

src/content/docs/browser/browser-monitoring/browser-pro-features/session-replay.mdx Show resolved Hide resolved
src/content/docs/browser/browser-monitoring/browser-pro-features/session-replay.mdx Outdated Show resolved Hide resolved
src/content/docs/browser/browser-monitoring/browser-pro-features/session-replay.mdx Show resolved Hide resolved
src/content/docs/browser/browser-monitoring/browser-pro-features/session-replay.mdx Outdated Show resolved Hide resolved
src/nav/browser.yml Outdated Show resolved Hide resolved
ally-sassman and others added 2 commits January 25, 2024 08:51
@ally-sassman ally-sassman merged commit ea8b8d2 into develop Jan 25, 2024
14 checks passed
@ally-sassman ally-sassman deleted the GA-session-replay branch January 25, 2024 18:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bradleycamacho bradleycamacho bradleycamacho left review comments

Assignees

@ally-sassman ally-sassman

Labels
content requests related to docs site content from_tw Identifies issues/PRs from Tech Docs writers
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ally-sassman @bradleycamacho