chore: Session Replay - GA #31182
chore: Session Replay - GA #31182
Conversation
marandaneto
requested review from
ioannisj,
a team,
sortafreel,
veryayskiy and
pauldambra
and removed request for
a team
![greptile-apps[bot]](https://avatars.githubusercontent.com/in/867647?s=60&v=4){kind=small}
There was a problem hiding this comment.
PR Summary
This PR removes the beta notification banner for Flutter session replay, indicating that the feature is now generally available (GA) rather than in beta.
- Removed the LemonBanner from
frontend/src/scenes/onboarding/sdks/sdk-install-instructions/flutter.tsxthat previously informed users about the beta status - The Flutter session replay feature is now considered production-ready, matching the status of session replay in other SDKs
- Installation and configuration instructions for Flutter session replay remain unchanged
1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile
📸 UI snapshots have been updated1 snapshot changes in total. 0 added, 1 modified, 0 deleted:
Triggered by this commit. |
📸 UI snapshots have been updated2 snapshot changes in total. 0 added, 2 modified, 0 deleted:
Triggered by this commit. |
📸 UI snapshots have been updated1 snapshot changes in total. 0 added, 1 modified, 0 deleted:
Triggered by this commit. |
📸 UI snapshots have been updated1 snapshot changes in total. 0 added, 1 modified, 0 deleted:
Triggered by this commit. |
📸 UI snapshots have been updated1 snapshot changes in total. 0 added, 1 modified, 0 deleted:
Triggered by this commit. |
|
Size Change: +25 B (0%) Total Size: 13.6 MB ℹ️ View Unchanged
|
posthog/tasks/usage_report.py
Outdated
| @@ -650,7 +650,7 @@ def get_teams_with_mobile_billable_recording_count_in_period(begin: datetime, en | |||
| WHERE min_first_timestamp BETWEEN %(begin)s AND %(end)s | |||
| GROUP BY session_id | |||
| HAVING (ifNull(argMinMerge(snapshot_source), '') == 'mobile' | |||
| AND ifNull(argMinMerge(snapshot_library), '') IN ('posthog-ios', 'posthog-android', 'posthog-react-native')) | |||
| AND ifNull(argMinMerge(snapshot_library), '') IN ('posthog-ios', 'posthog-android', 'posthog-react-native', 'posthog-flutter')) | |||
There was a problem hiding this comment.
This is where Flutter recordings will be billable
📸 UI snapshots have been updated2 snapshot changes in total. 0 added, 2 modified, 0 deleted:
Triggered by this commit. |
follow up PostHog/posthog-flutter#178
Important
👉 Stay up-to-date with PostHog coding conventions for a smoother review.
Problem
Changes
Does this work well for both Cloud and self-hosted?
How did you test this code?