Skip to content
This repository was archived by the owner on Dec 16, 2025. It is now read-only.

Fix status text for days without incidents being displayed wrong#38

Closed
snkaupe wants to merge 2 commits intomainfrom
fix-status-text
Closed

Fix status text for days without incidents being displayed wrong#38
snkaupe wants to merge 2 commits intomainfrom
fix-status-text

Conversation

@snkaupe
Copy link
Member

@snkaupe snkaupe commented Sep 5, 2024

Days without incidents where displayed with the status text "maintenance," not "operational."

This occurred due to us now using severity value 0 for the special maintenance severity value. The same value was used to initialise the property used to calculate a DailyStatus's overall severity value.

We now provide a field in the config file that allows users to specify the default severity value used for days without incidents.

…nance", not "operational"

Signed-off-by: Sebastian Kaupe <kaupe@gonicus.de>
@snkaupe snkaupe requested a review from joshuai96 September 5, 2024 07:20
Signed-off-by: Sebastian Kaupe <kaupe@gonicus.de>
@snkaupe snkaupe self-assigned this Sep 5, 2024
@snkaupe
Copy link
Member Author

snkaupe commented Dec 4, 2024

Changes adapted into new PR #45 .

@snkaupe snkaupe closed this Dec 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant