search

samvera / hyrax

Hyrax is a Ruby on Rails Engine built by the Samvera community. Hyrax provides a foundation for creating many different digital repository applications.
http://hyrax.samvera.org/
Apache License 2.0
183 stars 124 forks source link

Zotero Subscription is broken #806

Open mjgiarlo opened 7 years ago

mjgiarlo commented 7 years ago

Issue by cam156 Thursday Apr 20, 2017 at 12:01 GMT Originally opened as https://github.com/projecthydra/sufia/issues/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

  1. Register your application with zotero
  2. Edit your user profile
  3. click on Connect with Zotero
  4. You will see a failed job on resque with "User not found"
no-reply commented 3 years ago

can anyone test zotero integration?

crisr15 commented 3 years ago

@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?