-
Notifications
You must be signed in to change notification settings - Fork 22
Add EX Performance metrics doc #3055
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
Conversation
b08d768 to
a217b78
Compare
5e667b7 to
8026394
Compare
8026394 to
1b40d5e
Compare
to define a better mapping between the spec and implementation.
cefaa50 to
da2e5b8
Compare
5097f11 to
092f7fa
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds a new documentation file specifying performance metrics for the EX (Element X) application. The document outlines how performance metrics will be tracked using Sentry, including UX metrics, technical metrics, and additional contextual data to better understand application performance.
- Defines UX metrics (cold start, catch-up, notification to message, open a room, send a message) tracked as Sentry transactions
- Specifies additional data points to collect (device information, homeserver, database file sizes, account size)
- Outlines technical metrics tracked as Sentry spans and identifies future considerations for additional UX metrics
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
We made a review with @mxandreas: - `Cold start` --> `Cached Room list` - `Catch-up` --> `Up-to-date room list` - Remove the `What is it measuring?` column that has duplicated info.
stefanceriu
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is now implement for iOS in element-hq/element-x-ios#5020
lgtm 👍
to specify those metrics.
Rendered