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

Zotero Subscription is broken #806

Open
mjgiarlo opened this issue Apr 20, 2017 · 2 comments
Open

Zotero Subscription is broken #806

mjgiarlo opened this issue Apr 20, 2017 · 2 comments
Labels
Milestone

Comments

@mjgiarlo
Copy link
Member

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

  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 mjgiarlo added the bug label Apr 20, 2017
@mjgiarlo mjgiarlo added this to the Backlog milestone Apr 20, 2017
@no-reply
Copy link
Contributor

can anyone test zotero integration?

@crisr15
Copy link

crisr15 commented Apr 14, 2021

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants