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

Feat/dragndrop #1394

Merged
merged 21 commits into from Nov 22, 2022
Merged

Feat/dragndrop #1394

merged 21 commits into from Nov 22, 2022

Conversation

rupin90
Copy link
Contributor

@rupin90 rupin90 commented Nov 6, 2022

This is still in progress

  • user can drop any file on the library page. (I do have temporary code that shows the name of the file being dropped right on the page)
  • File upload APIs
  • Dragndrop UI
  • File upload progressbar

@vercel
Copy link

vercel bot commented Nov 6, 2022

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
omnivore-demo ✅ Ready (Inspect) Visit Preview Nov 22, 2022 at 4:04AM (UTC)
omnivore-prod ✅ Ready (Inspect) Visit Preview Nov 22, 2022 at 4:04AM (UTC)

@jacksonh
Copy link
Contributor

jacksonh commented Nov 9, 2022

Hey @rupin90 could we also add this to HomeFeedContainer.tsx so we can use it on the old library as-well as the new?

@rupin90
Copy link
Contributor Author

rupin90 commented Nov 9, 2022

Hey @rupin90 could we also add this to HomeFeedContainer.tsx so we can use it on the old library as-well as the new?

Yep yep, can do that today and will update this.

What about the file upload? API?

@Bad3r
Copy link

Bad3r commented Nov 17, 2022

Is the plan with this feature to parse the content of the PDF document? If that's the case; as far as I know, parsing PDF is going to be PIA.

There is going to be a need for a decent amount of tests with different types of PDFs and how to handle them. For example, some PDF files are not parsable without OCR.

What are the file formats that we want to accept? Is it just Pdf?

I would at least add EPUB to the list. Maybe also text files (.txt) and Markdown files (.md), and/or latex

Are we accepting multiple uploads at once?

A queue like system will be nice to have, but can be a separate feature. Also, adding the ability to send files through the API would be a nice addition.

@rupin90
Copy link
Contributor Author

rupin90 commented Nov 17, 2022

Is the plan with this feature to parse the content of the PDF document? If that's the case; as far as I know, parsing PDF is going to be PIA.

There is going to be a need for a decent amount of tests with different types of PDFs and how to handle them. For example, some PDF files are not parsable without OCR.

What are the file formats that we want to accept? Is it just Pdf?

I would at least add EPUB to the list. Maybe also text files (.txt) and Markdown files (.md), and/or latex

Are we accepting multiple uploads at once?

A queue like system will be nice to have, but can be a separate feature. Also, adding the ability to send files through the API would be a nice addition.

I think AFAIK it is going to be simple file upload. For now we are just doing one upload at a time.

Thank you for some nice to haves, definitely will consider to continue to evolve this feature.
For this PR we are adding drag and drop with single file upload.

I hope that answers all your questions. :)
Thank you.

@jacksonh
Copy link
Contributor

Hey @Bad3r we actually have a PDF viewer so we can render native PDFs, and allow users to highlight + add notes. Thanks for mentioning epub, we had actually just been talking about it, so its good to hear other people are interested.

@jacksonh
Copy link
Contributor

@Bad3r in terms of parsing the PDFs, we do use pdf.js on the backend to try and extract all the content of the PDF to create a search index for it. I think Logseq uses the same library.

@jacksonh
Copy link
Contributor

It looks like there is some z-order issue, i see the UI show up behind the cards

Screen Shot 2022-11-21 at 20 57 43

@rupin90
Copy link
Contributor Author

rupin90 commented Nov 21, 2022

It looks like there is some z-order issue, i see the UI show up behind the cards

Screen Shot 2022-11-21 at 20 57 43

should be good now, Having major internet issues 😓

@jacksonh
Copy link
Contributor

Nice! Works pretty well for me locally.

@jacksonh jacksonh merged commit 6afea97 into main Nov 22, 2022
@jacksonh jacksonh deleted the feat/dragndrop branch November 22, 2022 04:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants