Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix tzinfo missing attribute #1488

Merged
merged 1 commit into from
Apr 10, 2023
Merged

Conversation

steven9909
Copy link
Contributor

tzinfo does not have a localize attribute so it is replaced with timestamp in UTC

@ValueRaider
Copy link
Collaborator

ValueRaider commented Apr 10, 2023

Thanks for reading the developer guide, but I think this counts as an urgent bug fix so should go straight into main for a release. What do you think?

@steven9909
Copy link
Contributor Author

Yeah I think so. I can rebase the PR!

tzinfo does not have a localize attribute so it is replaced with timestamp in UTC
@steven9909 steven9909 changed the base branch from dev to main April 10, 2023 20:05
@ValueRaider ValueRaider merged commit 6d52cb6 into ranaroussi:main Apr 10, 2023
@steven9909 steven9909 deleted the fix_localize branch April 10, 2023 21:01
@ValueRaider
Copy link
Collaborator

ValueRaider commented Apr 11, 2023

This fixes Issues: #1268 #1387 #1467 #1478
May also fix (waiting for confirmation): #359 #939

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants