Skip to content

Conversation

@Jeehut
Copy link
Contributor

@Jeehut Jeehut commented Aug 15, 2025

⚠️ This should be merged together with the dashboard pages themselves: https://github.com/TelemetryDeck/web/pull/2541

@Jeehut Jeehut requested review from voidedmain and winsmith August 15, 2025 11:06
Copy link
Contributor

@voidedmain voidedmain left a comment

Choose a reason for hiding this comment

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

Really like the two docs! Good job! I especially love the questions you prepared for the user's to ask themselves, I think these are super helpful for them to get started.

@Jeehut
Copy link
Contributor Author

Jeehut commented Aug 18, 2025

@voidedmain I addressed all your feedback in 282030b, feel free to review the changes and let me know what you think.

@winsmith
Copy link
Contributor

winsmith commented Aug 19, 2025

Removing myself as a reviewer since I don't think I can add anything of value on top of what @voidedmain says. (I'm still reading with interest though)

@winsmith winsmith removed their request for review August 19, 2025 09:30
@Jeehut Jeehut merged commit 247c31c into main Aug 19, 2025
1 check passed
@Jeehut Jeehut deleted the feature/activation-retention branch August 19, 2025 12:08
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.

4 participants