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.

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

feat: add a list of default playlists to session replay #27371

Merged
merged 5 commits into from
Jan 10, 2025

Conversation

veryayskiy
Copy link
Contributor

The idea is to show as much value a possible for users when onboard / create new dashboard## Problem

At the moment we have 2 main groups of users:

  1. Users with low amount of traffic
  2. Users with high amount of of traffic

By “low”/”high traffic we mean ability to watch all recordings for the day (24h) within reasonable time, e.g. 30-60 mins.

Changes

Screenshot 2025-01-08 at 18 23 36

How did you test this code?

  1. Create a new account and set Session Replay during the onboarding process
    1.1. Click on Session Replay
    1.2. Click on Playlists

  2. Create a new project
    2.1. Click on Session Replay
    2.2. Click on Playlists

Aleksandr Veryayskiy added 5 commits January 8, 2025 17:47
The idea is to show as much value a possible for users when onboard / create new dashboard
[+] added to the team as it called on both - creation of the account and the new project
[*] moved the templates to helpers
@veryayskiy veryayskiy marked this pull request as ready for review January 10, 2025 08:05
@veryayskiy veryayskiy requested a review from pauldambra January 10, 2025 08:05
"filter_group": {"type": "AND", "values": [{"type": "AND", "values": []}]},
"filter_test_accounts": "true",
},
"description": "All session recordings longer than 1 minute",
Copy link
Member

Choose a reason for hiding this comment

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

description and filter don't match... do we want to change the description or the query?

Comment on lines +92 to +93
{"key": "level", "type": "log_entry", "value": ["error"], "operator": "exact"},
{"key": "message", "type": "log_entry", "value": "5", "operator": "gt"},
Copy link
Member

Choose a reason for hiding this comment

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

i wonder if this is really "has more than 5 log entries and at least one is an error"

but I didn't check and more than good enough to get started with!

Copy link
Member

@pauldambra pauldambra left a comment

Choose a reason for hiding this comment

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

I think we used to limit the total number of playlists people had... if so we can follow-up with a change to make sure we don't include these in the limit - but I don't remember if we still do that so not blocking on this! 🚀

@veryayskiy veryayskiy merged commit d9a2b7c into master Jan 10, 2025
94 checks passed
@veryayskiy veryayskiy deleted the feat/sr-default-playlists branch January 10, 2025 09:58
@raquelmsmith
Copy link
Member

We do limit it to 5 playlists on free plans! You'll need to update the check.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants