-
|
I'm test driving Ente Photos and I noticed that it is not able to import timezone-aware "date time original" EXIF values. Exiftool renders them as follows: I'd like to fix that. I assume that I'll need to apply a code change somewhere in https://github.com/ente-io/ente/blob/5806eb6e608dedc3713da1ebcab22f0cb87b7b23/web/packages/gallery/services/exif.ts. How do I validate that my fix won't break existing functionality? And where should I add a unit test validating that my code change handles the above case? |
Beta Was this translation helpful? Give feedback.
Replies: 1 comment
-
|
As mentioned in the contributing guide, instead of providing code, please explain the problem, and we will ourselves fix the problem you're facing in a holistic manner across clients instead of ad-hoc patches. I'm commenting just to save you effort - any non-trivial code PR is unlikely to get merged currently (primarily because we do not have maintainer bandwidth right now). |
Beta Was this translation helpful? Give feedback.
As mentioned in the contributing guide, instead of providing code, please explain the problem, and we will ourselves fix the problem you're facing in a holistic manner across clients instead of ad-hoc patches.
I'm commenting just to save you effort - any non-trivial code PR is unlikely to get merged currently (primarily because we do not have maintainer bandwidth right now).