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

Upload Multiple Files at Once #38

Open
marshalmiller opened this issue Dec 1, 2021 · 8 comments
Open

Upload Multiple Files at Once #38

marshalmiller opened this issue Dec 1, 2021 · 8 comments
Assignees

Comments

@marshalmiller
Copy link
Collaborator

A great feature would be if we could upload more than one file at a time. I know this is a major step, but I believe that this would be of great use to many.

@marshalmiller
Copy link
Collaborator Author

@jayeclark do you mind if I assign this to you or would you prefer to just get to it when you can? I just to make sure people know you're working on it.

@jayeclark
Copy link
Contributor

Sure!

I'm working on a couple freelance projects right now, but, can definitely get to this soon and I'd love to help.

@marshalmiller
Copy link
Collaborator Author

Thanks. No pressure. You've already helped so much.

@marshalmiller
Copy link
Collaborator Author

Hey, @jayeclark. I was wondering what method you planned to use to solve this? I'm sure you are super busy, and I have no desire to ask any more of you, but if you could outline the process here, perhaps I or someone else could piece it together.

@M-Faheem-Khan
Copy link
Contributor

I be willing to take this issue

@marshalmiller
Copy link
Collaborator Author

Thank you, @M-Faheem-Khan

marshalmiller added a commit that referenced this issue Aug 27, 2022
feat: Allow multi file uploads (#38)
@marshalmiller
Copy link
Collaborator Author

@M-Faheem-Khan This is a great advancement. There seems to be one issue, though. After the first document, the summary does not seem to update or complete. I guess the exception to that is the number of DOI references. But I think the program is stalling somewhere because you can still see the loading icon on the summary for them. Even in your screenshot, you can see the little loading circle next to the second link summary. Any help in figuring it out would be appreciated. For now, I have created a staging branch where I merged it. We can use that while we iron out any kinks.

@marshalmiller
Copy link
Collaborator Author

I think that we must first resolve issue #74 before this will work. @M-Faheem-Khan made the app upload multiple files. I moved that to the staging branch, but since the process times out before completing, it only even starts on the first file. Implementing Redis or something similar might be the only way to have this work efficiently.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants