removed column fetch_offset_timezone

openmeteo / enhydris

A database with a web interface for the storage and management of hydro/meteorological measurements and time series

GNU Affero General Public License v3.0

18 stars 11 forks source link

removed column fetch_offset_timezone #509

Closed Shrhawk closed 1 year ago

Shrhawk commented 1 year ago

Checklist:

[ ] All tests are passing
[ ] I have added any necessary documentation and/or release notes
[ ] I have followed the commit message guidelines and the "before submitting a pull request" guidelines
[ ] If I introduced (or abolished) any third-party library, I will write a short paragraph in this PR according to the guidelines
[ ] I have removed all obsoleted code

Shrhawk commented 1 year ago

Hello @aptiko Can you take care of migration file for the model changes I have done in the PR. I was not able to create the migration file.

Shrhawk commented 1 year ago

What was the problem with migrations? Doesn't ./manage.py makemigrations enhydris do it?

Django is not detecting any model change when I run the command. I also have tried without app name but then I get file write permission denied error.

aptiko commented 1 year ago

By the way, the commit message should also be Remove column fetch_offset_timezone (fixes #507) as described in "Git and GitHub" in confluence.

aptiko commented 1 year ago

Merged in 9211d7fd68698fbf6d8ca0024eaefbfcc321114e.