Skip to content

Conversation

@filippog
Copy link
Contributor

@filippog filippog commented Jan 17, 2025

  • I added CHANGELOG entry for this change.
  • Change is not relevant to the end user.

Changes

Add todo metrics to compact dashboard

Verification

Verified locally in Grafana

@filippog filippog force-pushed the compact-dashboard-todo branch from 27c4baa to be95743 Compare January 17, 2025 13:44
Signed-off-by: Filippo Giunchedi <fgiunchedi@wikimedia.org>
@filippog filippog force-pushed the compact-dashboard-todo branch from be95743 to dfd7164 Compare January 17, 2025 13:45
@filippog
Copy link
Contributor Author

@yeya24 WDYT?

yeya24
yeya24 previously approved these changes Feb 21, 2025
Copy link
Contributor

@yeya24 yeya24 left a comment

Choose a reason for hiding this comment

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

This makes sense to me. Thanks a lot!

Signed-off-by: Filippo Giunchedi <filippog@users.noreply.github.com>
@filippog
Copy link
Contributor Author

Tests failures are not related, and I've resolved the changelog conflict. Should be good to merge anytime @yeya24, thank you!

@filippog
Copy link
Contributor Author

@yeya24 hello, anything I should be doing on my end to get this merged? thank you !

@filippog
Copy link
Contributor Author

Can a maintainer please let me know if there is something else on my end on this PR before merging? Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants