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

Broken javascript, especially in file manager #896

Closed
hackartisan opened this issue Aug 12, 2016 · 3 comments
Closed

Broken javascript, especially in file manager #896

hackartisan opened this issue Aug 12, 2016 · 3 comments

Comments

@hackartisan
Copy link
Contributor

Descriptive summary

File uploader and File manager are both broken.

Javascript errors:
screen shot 2016-08-12 at 11 58 52 am

Making fileuploader the first require in app/assets/javascripts/curation_concerns/curation_concerns.js resolves the error and fixes the file uploader, but does not fix file manager.

Putting the filemanager requires first creates the following errors:
screen shot 2016-08-12 at 12 03 42 pm

The problem appears to have been introduced in samvera/hydra-editor@d38f1e7#diff-cf46d4dddad62fda63f883d306e44679 because when I pin to its parent the filemanager works and when I pin hydra-editor to that commit it is broken.

The file_manager es6 files are getting wrapped with what I believe @jcoyne says is some almond stuff, but they were already getting prepended with some other stuff I think from babel and it seems that the almond stuff is breaking them. https://gist.github.com/hackmastera/d6d26a85d55cbc938041cc1beac41f85 shows the result I'm getting in my browser.

Expected behavior

File manager should allow drag/drop of files and works

Actual behavior

No drag and therefore also no drop.

Steps to reproduce the behavior

  1. Pull master, do a new bundle install and engine cart generation
  2. Run the test app. create a work some some files
  3. navigate to file manager and attempt to drag something.

Related work

possibly related to #884?

@mjgiarlo
Copy link
Member

Pinging @tpendragon about this ticket too.

@tpendragon
Copy link
Contributor

@jcoyne I don't know anything about almond - can we just remove it, or should we be converting everything to work with it?

@tpendragon
Copy link
Contributor

tpendragon commented Aug 15, 2016

@HackMasterA Thank you so much for the detailed track-down. https://github.com/projecthydra/curation_concerns/compare/file_manager_issues fixes the JS part of the issue, but it looks like there's some ruby problems too that I'll try to identify tomorrow.

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

No branches or pull requests

3 participants