-
Notifications
You must be signed in to change notification settings - Fork 72
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
Custom metrics when viewing device #1804
Conversation
This is the more controversial part of #1796 and we don't have to merge it. I think it demos well. I think it is more useful than just showing our preferred ones. But not a hill I'll die on, made the PR as much to save the generic code since we may want it on the Health view :D |
I'd be happy to merge this in if we are to combine it with an opt-in in the Product settings to show all or 'extended' metrics. (off by default) Long term, I'd like to add a settings icon (cog) to the bottom right which gives you a way to customize what metrics you see on the device page. |
Yeah, I am on board with some kind of product-wide setting to control what shows up. I'll noodle on something. |
+1 for me as well! |
I love this! I had intended to add this to the health tab, thank you! |
Ah, sorry, didn't see that it hadn't been pushed to the PR yet 😅 I know its a little none standard, I'm wondering if we remove the border and title and just have the stats. We can also move the 'last updated' next to the period select buttons? |
cb748b0
to
56e6918
Compare
Did I forget to commit and just pushed air? Maybe a little.. |
56e6918
to
ac7d446
Compare
ac7d446
to
8c0c661
Compare
Adds custom metrics for display: