Skip to content

use the v4 api endpoint - #9

Merged
alainmeier merged 6 commits into
BlockScore:masterfrom
ryanahall:rhall/use_v4_endpoint
Mar 3, 2015
Merged

use the v4 api endpoint#9
alainmeier merged 6 commits into
BlockScore:masterfrom
ryanahall:rhall/use_v4_endpoint

Conversation

@ryanahall

Copy link
Copy Markdown
Contributor

@alainmeier thanks for the quick merge on the last pr.

Like I mentioned earlier, here's the v4 endpoing version. I've renamed verifications to people and question_set to question_sets to be consistent with the ruby library. I've also converted all hard tabs to soft tabs with tabstop set to 2 characters as per the last pr. Finally, I changed the minor version number because these are breaking changes, though you may want to change the major version since the api endpoint was updated.

Let me know what you think.

Thanks,
Ryan

@ryanahall ryanahall changed the title Rhall/use v4 endpoint use the v4 api endpoint Mar 3, 2015
@alainmeier

Copy link
Copy Markdown
Member

This is great, will test it soon. In the meantime, can you please make the version 4.0.0 for the package? We like the major version number to correspond with the API version.

@alainmeier

Copy link
Copy Markdown
Member

👍

alainmeier added a commit that referenced this pull request Mar 3, 2015
@alainmeier
alainmeier merged commit 4200583 into BlockScore:master Mar 3, 2015
@ryanahall

Copy link
Copy Markdown
Contributor Author

Thanks again @alainmeier! Do you have any plans to upload 4.0.0 to pypi?

@ryanahall
ryanahall deleted the rhall/use_v4_endpoint branch March 3, 2015 20:38
@alainmeier

Copy link
Copy Markdown
Member

Definitely, hoping to get that pushed in a couple hours

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants