We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Issue by cam156 Thursday Apr 20, 2017 at 12:01 GMT Originally opened as samvera-deprecated/sufia#3163
In Sufia 7.2 and later version (I think) the connection between Sufia and Zotero is broken. The job expects and id and the code is passing a class: https://github.com/projecthydra/sufia/blob/master/app/controllers/api/zotero_controller.rb#L25
The controller should pass the expected parameter to the job
The controller passes a job instead of a string id that is expected, so when the job does a find it gets an error about the user being missing.
The text was updated successfully, but these errors were encountered:
can anyone test zotero integration?
Sorry, something went wrong.
@no-reply I tested this on https://uindy.hykucommons.org/concern/generic_works/a5d072c0-d529-4a6d-889f-1f3d7a65eaad?locale=en and was able to use the Chrome extension to send the metadata to my Zotero account. Hyku Commons is running Hyrax 2.9 (Hyku 3.0?). Does this need to be tested on a Hyrax 3 app?
No branches or pull requests
Issue by cam156
Thursday Apr 20, 2017 at 12:01 GMT
Originally opened as samvera-deprecated/sufia#3163
Descriptive summary
In Sufia 7.2 and later version (I think) the connection between Sufia and Zotero is broken.
The job expects and id and the code is passing a class: https://github.com/projecthydra/sufia/blob/master/app/controllers/api/zotero_controller.rb#L25
Expected behavior
The controller should pass the expected parameter to the job
Actual behavior
The controller passes a job instead of a string id that is expected, so when the job does a find it gets an error about the user being missing.
Steps to reproduce the behavior
The text was updated successfully, but these errors were encountered: