Skip to content

Conversation

Wind-stormger
Copy link

Adds only a minimal amount of code, allowing manual setting of the timezone.

Similar to this PR, but it's closed. #634

Signed-off-by: Wind-stormger <storm-wind@foxmail.com>
@MrEbbinghaus
Copy link

Same issue as in: #715
See my comment: #715 (comment)

@projectgus
Copy link
Contributor

Thanks for submitting this PR. Everyone seems to agree MicroPython should solve this problem, but the best place is probably not in the ntptime module (as NTP itself doesn't know about timezones, and numeric Unix timestamps are generally always in UTC.).

I've started a Discussion here to talk about the possible ways forward for this:
https://github.com/orgs/micropython/discussions/12378

Please feel free to reply there with any other thoughts or suggestions.

@projectgus projectgus closed this Sep 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants