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

Add cloud files not working #2804

Open
julesies opened this issue Mar 12, 2018 · 16 comments
Open

Add cloud files not working #2804

julesies opened this issue Mar 12, 2018 · 16 comments
Labels
bug feedback needed help requested Something about this issue/PR requires help from others In Discussion
Milestone

Comments

@julesies
Copy link

julesies commented Mar 12, 2018

Several Cloud Related bugs have been reported and I can't tell what is a Nurax set up issue, a hyrax issue, or a Browse-Everything issue. I thought I'd report them all here and write up to the appropriate place as needed. I know there are BE PRs that may address some of these.

  1. Import from google drive prompts user "the app isn't verified" Cannot Connect to Google Drive when selecting 'add cloud files' samvera-labs/nurax-pre2023#217
  2. Import from dropbox allows authenticated to client via hyrax but then prompts users to enter a code into nurax: Dropbox prompts you to add code to Nurax when selecting 'add cloud files' samvera-labs/nurax-pre2023#230
  3. Import from box successfully imports but strips the file extension and so the user downloads a file that is confusing/generic. Import from box strips file extension samvera-labs/nurax-pre2023#233
@julesies
Copy link
Author

julesies commented Apr 5, 2018

@mbklein @jrgriffiniii should any of these 3 clients work with Hyrax 2.1.0beta2 that was released yesterday?

@jrgriffiniii
Copy link
Contributor

@julesies samvera/browse-everything#202 is required for FileSystem browsers on Valkyrie repositories, so that may also no longer function on more recent Hyrax releases.

I'll need to test the GoogleDrive client, but I had it functioning for 2.0.0. I've been progressing with Dropbox on https://github.com/jrgriffiniii/browse-everything/tree/hotfix-dropbox and can return to this later today.

@vantuyls
Copy link

vantuyls commented Apr 9, 2018

I'm still looking for a little clarity here - @jrgriffiniii, do we expect BE to work for hyrax currently (or in the impending release)? we are hoping not to block the release based on upstream (ie BE) problems, but if the issues we're seeing are not BE related, we'll need to fix them before release. Can you help us tease this apart?

@jrgriffiniii
Copy link
Contributor

@vantuyls this is definitely an upstream problem, and I would please recommend that any impending Hyrax 2.x releases not be blocked by Dropbox issues in browse-everything.

@vantuyls
Copy link

vantuyls commented Apr 9, 2018

thanks @jrgriffiniii

@julesies
Copy link
Author

julesies commented Apr 9, 2018

@jrgriffiniii what about box and google drive? i believe box works, but there is a hyrax bug that strips out the file extension. and i'm not sure the status of google drive.

@no-reply no-reply added the bug label Jun 3, 2018
@no-reply no-reply added this to the 2.x series milestone Jun 3, 2018
@vantuyls
Copy link

@jrgriffiniii can you give us an update on this issue and a status of any work that is ongoing or planned? @no-reply

@no-reply
Copy link
Contributor

Anyone know where we are on this one? Did various BrowseEverything upgrades resolve any portion of this?

@rjkati rjkati self-assigned this Jan 28, 2021
@rjkati
Copy link

rjkati commented Jan 28, 2021

In Hyrax 3.0.0 rc3, only Box can be selected as a cloud provider.
box

I am able to connect my Box account and select a file to upload ("Welcome to Box.pdf"). When I click "Submit", the "Submit" button displays a "Loading..." message and a spinning circle appears. The file does not upload and clicking the "Cancel" button does nothing.
spinnybox

This occurs in both Chrome 88.0.4324.104 and Firefox 85.0. @jlhardes should this be a new ticket?

@jlhardes
Copy link
Contributor

I think nurax-dev isn't set up to connect to various cloud providers, only to Box. And that setup might not be complete either. @jrgriffiniii, it looks like all services should be working (https://github.com/samvera/browse-everything) - is this more about inquiring with nurax-dev regarding what is configured for cloud upload?

@cudevmaxwell
Copy link
Contributor

@rjkati We're also seeing the same behaviour for the file system provider. browse-everything 1.1.2, hyrax master branch head. Might need to be a new issue at the browse-everything repo?

@wickr
Copy link
Contributor

wickr commented Mar 19, 2021

We're seeing this as well on ScholarsArchive, on Hyrax 2.9.4.

osulp/Scholars-Archive#2215

The plugin works and passes the file location to Hyrax, but won't close the modal. In our ticket is a workaround, which is to manually remove the browse-everything elements after submitting. After doing that, you can submit the file.

@jlhardes
Copy link
Contributor

This looks like it might be something to report over in browse-everything for a fix?

@jlhardes
Copy link
Contributor

Rutgers' recent implementation of browse-everything didn't have any issues with any cloud providers. We think recent work in browse-everything by @no-reply might have addressed the issue described here, can you verify?

@jlhardes
Copy link
Contributor

Rutgers is using Hyrax 2.x and the latest browse-everything release does not include @no-reply's recent changes so we might need to check in with the community to see if we have an issue to work on here still for browse-everything with Hyrax 3.x.

@jlhardes jlhardes added the help requested Something about this issue/PR requires help from others label Aug 13, 2021
@fritzfreiheit
Copy link
Contributor

fritzfreiheit commented Mar 2, 2022

See: samvera/browse-everything#376 (Add cloud files popup ignores cancel button)

@rjkati rjkati removed their assignment Sep 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug feedback needed help requested Something about this issue/PR requires help from others In Discussion
Projects
None yet
Development

No branches or pull requests

9 participants