dataquest-dev / DSpace

(Official) The DSpace digital asset management system that powers your Institutional Repository
https://wiki.lyrasis.org/display/DSDOC7x/
BSD 3-Clause "New" or "Revised" License
1 stars 2 forks source link

Update sourceversion.py #694

Closed MajoBerger closed 1 month ago

MajoBerger commented 2 months ago
Phases MP MM MB MR JM Total
ETA 0 0 0 0 0 0
Developing 0 0 0 0 0 0
Review 0 0 0 0 0 0
Total - - - - - 0
ETA est. 0
ETA cust. - - - - - 0

Problem description

Reported issues

Not-reported issues

Analysis

(Write here, if there is needed describe some specific problem. Erase it, when it is not needed.)

Problems

(Write here, if some unexpected problems occur during solving issues. Erase it, when it is not needed.)

MajoBerger commented 1 month ago

That is true, it is not good like this. Would it be better to go without the timezone? Then we miss some information. Or should we just do it with timezone and without offset? That could be always correct, only not appropriate to OUR timezone.

MajoBerger commented 1 month ago

https://docs.python.org/3/library/zoneinfo.html#module-zoneinfo or we can do this and make a condition for python version 3.9+

MajoBerger commented 1 month ago

use conditional import for timezone