Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add timezone setting #3652

Merged
merged 1 commit into from
Oct 12, 2020
Merged

Add timezone setting #3652

merged 1 commit into from
Oct 12, 2020

Conversation

frafra
Copy link
Contributor

@frafra frafra commented Sep 27, 2020

Fix #3651.

@willemarcel
Copy link
Contributor

willemarcel commented Sep 28, 2020 •
edited
Loading

@frafra The backend is not prepared to deal with time zones other than UTC, as it's not saving dates on the database with time zone information. I believe adding that setting will cause problems.

@frafra
Copy link
Contributor Author

frafra commented Sep 29, 2020

Thanks for clarifying. Ok, so should be just set TZ=UTC as default environmental variable inside the Dockerfile just to mute the warning?

@willemarcel
Copy link
Contributor

Yes, adding that command to the Dockerfile seems good to me

@frafra frafra force-pushed the add-tz branch 2 times, most recently from 8367328 to c3a6569 Compare September 30, 2020 20:42
@willemarcel willemarcel merged commit 8536b2e into hotosm:develop Oct 12, 2020
@frafra frafra deleted the add-tz branch October 12, 2020 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
scope: infrastructure
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Backend raises a UserWarning: No timezone specified
2 participants
@frafra @willemarcel