Done.
Remove extra blank line.
RESOLVED in CODE WALKTHROUGH.
Keep this blank line. Separate declarations from code.
This should be the first if and the one above should just be an else (no if).
Align these with first 2 #defines.
updated
Not sure why this code is here - comment says remove so please remove these 2 lines of code.
RESOLVED IN CODE WALKTHROUGH
Remove if not needed.
Update header
Merge markers need to be removed.
Remove blank line.
F32 needs 0.0F init value.
Done
Added.
Add disinfectCancelReqID to header. Also argument stop.
Removed for Phase 1B? Needs comment if so.
Should 250 be #define?
No.
I think this check is for over-fill. Counter is for illegal level sensor readings. Over-fill check should have it's own counter (or use new persistence style) and we should reset that separate counter in the "else" condition.
Fixed. Thanks!
Got a suggestion or an issue? Discuss it on the Atlassian Community or visit Atlassian Support for other options.
