Skip to content

Conversation

@TiVoHomeUser
Copy link
Contributor

@TiVoHomeUser TiVoHomeUser commented Apr 18, 2021

Fixes #124 & #127.

  • This PR is compliant with the other contributing guidelines as well (if not, please describe why).

  • I have thoroughly tested my contribution.

  • Should this code require changes to documentation I will contribute those to https://github.com/ThingPulse/docs.

  • Fix for broken sleep mode

  • Soft sleep mode

  • Fix for 12/24 hr displays not changing

Fix for missing reset sleep timer for screen press.
New option of Background light off sleep also delays download of weather data
Restore Settings.h
Clean up and getting local repository in sync with gitHub
Fix for 24/12 hr time display in
- ForecastDetail
- ForecastTable
- drawAstronomy
Copy link

@SillyHats SillyHats left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this. For 12h formatting preferences I changed %02d to %2d to remove leading zeros in the hours in the times on my board.

@TiVoHomeUser
Copy link
Contributor Author

Nice 8:00 PM is better then 08:00 PM for the 12 HR format

Copy link
Contributor Author

@TiVoHomeUser TiVoHomeUser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix for Sleep mode and added Light off sleep mode with screen press wake
Change for 12/24 hr format in forcast

@TiVoHomeUser TiVoHomeUser marked this pull request as ready for review May 4, 2021 20:42
@TiVoHomeUser TiVoHomeUser marked this pull request as draft May 4, 2021 20:44
@TiVoHomeUser TiVoHomeUser reopened this May 4, 2021
@TiVoHomeUser TiVoHomeUser marked this pull request as ready for review May 4, 2021 20:48
@TiVoHomeUser TiVoHomeUser marked this pull request as draft May 4, 2021 20:49
@TiVoHomeUser TiVoHomeUser marked this pull request as ready for review May 4, 2021 22:29
@TiVoHomeUser
Copy link
Contributor Author

Ready

@TiVoHomeUser TiVoHomeUser marked this pull request as draft May 5, 2021 21:08
@TiVoHomeUser TiVoHomeUser marked this pull request as ready for review May 7, 2021 19:05
@TiVoHomeUser
Copy link
Contributor Author

Fixed conflicts any thing more I need to do?

@marcelstoer
Copy link
Member

😢 😞 so sorry I missed to review this in due time. Hope I'll find some time next week.

@marcelstoer marcelstoer merged commit 5b34146 into ThingPulse:master Dec 25, 2021
@TiVoHomeUser TiVoHomeUser deleted the time_period branch December 26, 2021 00:43
@TiVoHomeUser TiVoHomeUser restored the time_period branch January 30, 2022 01:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

12 hour setting on clock but not on forecasts

3 participants