Skip to content
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

Remove or explain "Current Alarms" section when empty #1734

Open
elinol opened this issue Jan 7, 2025 · 2 comments
Open

Remove or explain "Current Alarms" section when empty #1734

elinol opened this issue Jan 7, 2025 · 2 comments

Comments

@elinol
Copy link
Contributor

elinol commented Jan 7, 2025

When Current Alarms list is empty we should remove that section from device health in UI, alternatively tell that list is empty.
It looks a bit odd with a header without any content.

image

@joshk
Copy link
Collaborator

joshk commented Jan 8, 2025

One idea, if there are no active alarms we should change the text to No active alarms with a link below it that reads Learn how to setup Device alarms that links to some nerves_hub_link code/docs/guide?

@elinol
Copy link
Contributor Author

elinol commented Jan 10, 2025

I'm not sure there is any docs updated enough to link to, can only find this: https://github.com/nerves-hub/nerves_hub_link?tab=readme-ov-file#alarms

let's start with No active alarms?

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

No branches or pull requests

2 participants