FalckJoshua / DIT113-System-Development-Locus-Imperium

Other
0 stars 0 forks source link

added alert for temperature #19 - [merged] #93

Closed FalckJoshua closed 7 months ago

FalckJoshua commented 1 year ago

In GitLab by @cjoshua on May 2, 2023, 13:41

Merges 19-give-an-alert-if-the-acceptable-temperature-is-exceeded -> main

What does this MR do?

This merge adds visual alertness when the temperature exceeds the value assigned in settings.

Related issues

[Related to #13] [Closes #19]

Acceptance criteria met

FalckJoshua commented 1 year ago

In GitLab by @cjoshua on May 2, 2023, 13:41

requested review from @carlosca

FalckJoshua commented 1 year ago

In GitLab by @cjoshua on May 2, 2023, 16:27

requested review from @willand and removed review request for @carlosca

FalckJoshua commented 1 year ago

In GitLab by @willand on May 2, 2023, 18:25

approved this merge request

FalckJoshua commented 1 year ago

In GitLab by @willand on May 2, 2023, 18:25

Looks good, follows the same principles set with the previous commits. Merging.

FalckJoshua commented 1 year ago

In GitLab by @willand on May 2, 2023, 18:25

mentioned in commit 6d86798019f24b32474e5d1f5c4405d1d5310f93