mrmn2 / PdfDing

Minimal and selfhosted PDF manager and viewer offering a seamless user experience on multiple devices.
GNU General Public License v3.0
27 stars 1 forks source link

Bulk import pdf library: check if exists #10

Open bairhys opened 3 days ago

bairhys commented 3 days ago

Hi @mrmn2, I tested the bulk import function and it is excellent, a lot easier to import! Thanks for adding it!

A nice to have minor addition would be if an uploaded PDF already exists, it shows the user and skips uploading it. At the moment, it looks like it appends a string to the filename of uploaded duplicate. The appended string does make it easier to identify duplicates to delete though.

Off topic but I also tested inverted colours and that works great too!

mrmn2 commented 3 days ago

Hi @bairhys , thanks for the testing. Informing the user about skipped file might be a bit harder to implement. Would it work to add an option to skip files if a file with the same name already exists?

bairhys commented 1 day ago

Yeah I think a skip matching filenames option would work great