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.

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

[Enhancement] API to set photo upload date #984

Closed
endotronic opened this issue Apr 30, 2021 · 4 comments · Fixed by #1591
Closed

[Enhancement] API to set photo upload date #984

endotronic opened this issue Apr 30, 2021 · 4 comments · Fixed by #1591
Labels
enhancement New feature or request good first issue Good for newcomers Low Priority Low priority issues

Comments

@endotronic
Copy link

Hello! Thanks for all the awesome work. I love Lychee so far.

I'm working on a script to import photos into Lychee, and right now, all imported photos have the same upload date (now). I would like to be able to set the upload date to a different date, which I happen to have data for. Note that this is still different than the takestamp. I have no issue with that, but that is not what I'm looking for.

Thanks!

@ildyria ildyria added enhancement New feature or request good first issue Good for newcomers Low Priority Low priority issues labels Apr 30, 2021
@ildyria
Copy link
Member

ildyria commented Apr 30, 2021

Hi thank you for your suggestion. Currently the only way to edit this is by editing directly the database. :(

That being said, it could be interesting to add it as a "service" but as this is a pretty isolated case, I would suggest you to implement it and propose it as a Pull Request.
The main devs of the team (@kamil4 , @tmp-hallenser , & me) are quite is quite busy at the moment: Lychee does not pay the bills. ;) I am working a full refactoring of the front-end as a result, it is very unlikely that any of us has the time to work on such suggestion (sorry for the brutally honest answer).

@endotronic
Copy link
Author

No worries; I figured this was the case. I'll see if I can add it to the parameters for Photo::add since I always know it at add time. This is a personal project for me, though, so I don't have a lot of time either.

Since you are working on the frontend, I opened another request I have that may be in the space you are working. I imagine it is small: #987

@kamil4
Copy link
Contributor

kamil4 commented May 2, 2021

Out of curiosity, what is your use case? I've been meaning to make the takestamp editable for a long time, but I never thought that some might want to do it with the upload date?

@endotronic
Copy link
Author

I'm scraping photos from a forum for a project, and Lychee is a great way to manage them and search through them. In this scenario, I would prefer to use uploaded date to reflect the date it was posted to the forum. Takestamp also makes sense, but if the forum isn't stripping EXIF data (I haven't checked), then there is potential value in keeping that true to the original.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers Low Priority Low priority issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants