-
Notifications
You must be signed in to change notification settings - Fork 6
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
Add json-api-client to this repo #186
Conversation
4bf4d65
to
51476f2
Compare
If we merge this, we can deprecate https://github.com/zooniverse/json-api-client. zooniverse/Panoptes-Front-End#6369 is a feature branch that's set up to test this version of the client. |
fc5523a
to
e2ad22e
Compare
The JSON-API client code here is from v7.0.2. |
e2ad22e
to
710567e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested this at https://pr-6369.pfe-preview.zooniverse.org/ throughly. Signed in/out, made classifications, posted/edited to Talk, created a project, edited a project/workflow/task. I think this is working well, but let me know if there are any user actions I've neglected to test.
So I think this is good to go! 👍
Move the `json-api-client` code into `lib/json-api-client` and require it from there. Remove `json-api-client` as a dependency.
710567e
to
66e70a4
Compare
Move the
json-api-client
code intolib/json-api-client
and require it from there. Removejson-api-client
as a dependency.