search

samvera-deprecated / sufia

[DEPRECATED] Sufia: a fully featured, flexible Samvera repository front-end.
http://sufia.io/
Other
111 stars 78 forks source link

Zotero Subscription is broken #3163

Open carolyncole opened 7 years ago

carolyncole commented 7 years ago

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"
mjgiarlo commented 7 years ago

I imagine we'll want this in Hyrax as well.