-
Notifications
You must be signed in to change notification settings - Fork 161
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
OSW Weather emulator support #376
Conversation
+The OSW weather app is now supported also when using the emulator. +Documentation update
@@ -430,19 +483,21 @@ int OswAppWeather::getPrevDay() { | |||
|
|||
|
|||
void OswAppWeather::setup() { | |||
OSW_LOG_I("Setup "); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you cleanup these debug-prints? Or remove them from the release-build via OSW_LOG_D
?
@@ -6,15 +6,23 @@ OswAppWeatherEncoder::OswAppWeatherEncoder() {} | |||
bool OswAppWeatherEncoder::setUpdate(OswAppWeather::weather_update_t update) { | |||
bool update_ok = true; | |||
if(update.temp > 99 || update.temp < -99 ) { | |||
update_ok = false; | |||
OSW_LOG_I("ERROR TEMP"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-> OSW_LOG_E
😉
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
(or OSW_LOG_W
if you can still continue with the primary function)
Cleanup debug print
Error print
Warnings
Errors and warnings
Warnings
Hmmm, this is still a draft... |
I'll cleanup and finalize this PR in #382, as the author did not authorize modifications on its fork before merge. As I cannot change this PRs source, I'll close it instead. |
This pull request introduces changes to enable emulator support in the application OSW Weather. The key enhancement is the ability of the emulator to load a JSON file containing a valid API response from OpenWeather.
Changes:
Added emulator support to OSW Weather
Documentation update