-
-
Notifications
You must be signed in to change notification settings - Fork 4.6k
feat(dashboards): scaffold mobile session health module #107167
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(dashboards): scaffold mobile session health module #107167
Conversation
static/app/views/dashboards/utils/prebuiltConfigs/mobileSessionHealth.ts
Outdated
Show resolved
Hide resolved
static/app/views/dashboards/utils/prebuiltConfigs/mobileSessionHealth.ts
Show resolved
Hide resolved
static/app/views/dashboards/utils/prebuiltConfigs/mobileSessionHealth.ts
Outdated
Show resolved
Hide resolved
…ic-mobile-session-health-module
There was a problem hiding this comment.
Choose a reason for hiding this comm F43F ent
The reason will be displayed to describe this comment to others. Learn more.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.
| ], | ||
| }, | ||
| ], | ||
| 4 |
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.
Dashboard layout has gap due to incorrect y position
Medium Severity
THIRD_ROW_WIDGETS is positioned at y=4 but SECOND_ROW_WIDGETS ends at row 2 (y=1 with h=2). This creates an empty row 3 in the dashboard layout. The y parameter passed to spaceWidgetsEquallyOnRow should be 3 instead of 4 to ensure continuous widget placement without gaps.
Scaffold basic mobile health module with what is currently available in dashboards. Visibility of the dashboard version is behind a flag
