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(replay): add "what to watch" screen #25717

Merged
merged 42 commits into from
Oct 22, 2024
Merged

Conversation

raquelmsmith
Copy link
Member

@raquelmsmith raquelmsmith commented Oct 21, 2024

Problem

People sometimes find it difficult to know where to start with replays. Our best advice is to think of a product area you want to focus on, and then drill down form there.

We can productize that advice by giving people a jumping-off point, suggestions for areas they can look into.

Changes

Adds a "what to watch" screen with "templates" suggesting things they might want to watch.

Screen.Recording.2024-10-21.at.12.44.49.PM.mov

TODO:

  • Figure out why feature flags filtering doesn't work
  • Put this behind a flag so I can test it live
  • Add spacing above add button
  • Figure out why the variables don't hide when I tell them to hide...

👉 Stay up-to-date with PostHog coding conventions for a smoother review.

Does this work well for both Cloud and self-hosted?

How did you test this code?

manually 🙈

@raquelmsmith raquelmsmith requested a review from a team October 21, 2024 19:58
@raquelmsmith raquelmsmith changed the title stub out a templates screen feat(replay): add "what to watch" screen Oct 21, 2024
@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

  • chromium: 0 added, 2 modified, 0 deleted (diff for shard 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

  • chromium: 0 added, 2 modified, 0 deleted (diff for shard 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

Copy link
Member Author

Choose a reason for hiding this comment

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

lmk if you have ideas for other templates. I want to keep the list short & easy to parse, but we can decide which ones are best.

Copy link
Member

Choose a reason for hiding this comment

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

we could add images copied from dashboard templates to the cards to make it look more like the dashboard templates list

(although arguably that's very visually busy and could be hard to parse)

Screenshot 2024-10-21 at 22 11 29

Copy link
Member

Choose a reason for hiding this comment

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

we've got "purchase flow" and "pricing page" but I think they're really the same?

could possibly lose the B2B/B2C split too but I guess then it's just one big lump

Copy link
Member

Choose a reason for hiding this comment

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

could add a rage click one?

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh rageclick is a good idea, will add! What about deadclicks - didn't you have some event sending for that?

Copy link
Member Author

Choose a reason for hiding this comment

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

we could add images copied from dashboard templates

I very much dislike that UI 😅 🙈 but I will experiment with some icons.

we've got "purchase flow" and "pricing page" but I think they're really the same?

I think they're different for b2b and b2c/ecommerce - one is when someone is viewing eg saas pricing before signup, the other is when someone is purchasing something directly (eg a pair or earrings or a hammer). I think they should be different templates.

could possibly lose the B2B/B2C split too but I guess then it's just one big lump

I'm gonna keep the split for now to (hopefully) make it easier for people to find the things that are relevant to them. But we can see what people use and go from there.

Copy link
Member Author

Choose a reason for hiding this comment

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

Added some icons

image

Comment on lines 126 to 132
{
key: 'scattershot',
name: 'Scattershot',
description: 'Watch all recent replays, and see where users are getting stuck.',
variables: [],
categories: ['More'],
},
Copy link
Member

Choose a reason for hiding this comment

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

could replace this one with "by activity score"

if visual regression ever lets me merge #25701

i guess this is two points...
one let's have activity score
two do we need this scattershot one? since it's the equivalent of "not choosing a template"

no strong feeling but if you're looking to keep the list short

Copy link
Member

Choose a reason for hiding this comment

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

if we keep scattershot, it should actively set the ordering to recent
if someone has already changed sorting away from recent then they click on scattershot it doesn't reset

quite an edge case since not many folk change sorting

@@ -0,0 +1,147 @@
import { ReplayTemplateType } from '~/types'

export const replayTemplates: ReplayTemplateType[] = [
Copy link
Member

Choose a reason for hiding this comment

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

could have a "sessions with errors"

any $exception event or console_error_count > 0

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.

some typescript faff to solve but otherwise 🚢

(lots of comments only reflects being excited about the feature :))

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

  • chromium: 0 added, 2 modified, 0 deleted (diff for shard 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

  • chromium: 0 added, 2 modified, 0 deleted (diff for shard 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@pauldambra pauldambra force-pushed the feat/replay-templates branch from 0cde296 to 6b69dd5 Compare October 22, 2024 10:45
@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

4 snapshot changes in total. 0 added, 4 modified, 0 deleted:

  • chromium: 0 added, 4 modified, 0 deleted (diff for shard 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

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.

couple of comments

but stamp so you can keep moving in case TZ overlap gets in the way 🙈

frontend/src/types.ts Outdated Show resolved Hide resolved
@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

4 snapshot changes in total. 0 added, 4 modified, 0 deleted:

  • chromium: 0 added, 4 modified, 0 deleted (diff for shard 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

  • chromium: 0 added, 2 modified, 0 deleted (diff for shard 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

  • chromium: 0 added, 2 modified, 0 deleted (diff for shard 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

  • chromium: 0 added, 2 modified, 0 deleted (diff for shard 1)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@raquelmsmith raquelmsmith force-pushed the feat/replay-templates branch from 9d69bcd to a68c1a0 Compare October 22, 2024 17:54
@raquelmsmith raquelmsmith enabled auto-merge (squash) October 22, 2024 17:54
Copy link
Contributor

Size Change: +36 B (0%)

Total Size: 1.14 MB

ℹ️ View Unchanged
Filename Size Change
frontend/dist/toolbar.js 1.14 MB +36 B (0%)

compressed-size-action

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

  • chromium: 0 added, 2 modified, 0 deleted (wasn't pushed!)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

@raquelmsmith raquelmsmith merged commit 45257c1 into master Oct 22, 2024
92 of 93 checks passed
@raquelmsmith raquelmsmith deleted the feat/replay-templates branch October 22, 2024 18:12
@raquelmsmith
Copy link
Member Author

automerged without full green checks, but only because of flapping, so it's fine

@posthog-bot
Copy link
Contributor

📸 UI snapshots have been updated

2 snapshot changes in total. 0 added, 2 modified, 0 deleted:

  • chromium: 0 added, 2 modified, 0 deleted (wasn't pushed!)
  • webkit: 0 added, 0 modified, 0 deleted

Triggered by this commit.

👉 Review this PR's diff of snapshots.

fuziontech added a commit that referenced this pull request Oct 22, 2024
* master:
  fix(replay-templates): show event selector button, diff mobile filter (#25739)
  feat(replay): add "what to watch" screen (#25717)
  fix(events): Fix PostHog property filtering (#25732)
  fix(api): Apply org/team key scoping more narrowly (#25634)
  feat(Experiments): UX feedback from private beta of no-code experiments. (#25694)
  fix: playlist should track current session id (#25730)
  chore(data-warehouse): Set the stripe version when listing objects (#25729)
  chore(data-warehouse): Added extra props for hubspot syncs (#25728)
  revert: perf: Speed up filtering persons pt 2 (#25731)
  chore(deps): Update posthog-js to 1.174.3 (#25724)
  perf: Speed up filtering persons pt 2 (#25718)
  feat(alerts): toggle alert threshold lines (#25700)
  feat: watch next and sort different (#25701)
  feat(cdp): add microsoft teams template (#25652)
timgl pushed a commit that referenced this pull request Oct 25, 2024
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Paul D'Ambra <[email protected]>
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