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

Tokens API, improved testing and small extra utils #5

Merged
merged 6 commits into from Dec 1, 2015

Conversation

nicrioux
Copy link
Contributor

@nicrioux nicrioux commented Dec 1, 2015

Hi,
in case your interested, I added the Tokens API, improved the testing (still working on it) and added some small utilities like: delete_all/cancel_all/all and an efficient count using include[]=total_count&limit=0 params.

@nicrioux
Copy link
Contributor Author

nicrioux commented Dec 1, 2015

Forgive the double commits, I'm still figuring out git!

@robconery
Copy link
Contributor

This looks good - do you want to run another PR? Or are you happy with where you're at? Also, if you want to open an issue then push a PR for that issue it might help.

Either way, grateful for the help here thanks!

robconery added a commit that referenced this pull request Dec 1, 2015
Tokens API, improved testing and small extra utils
@robconery robconery merged commit 84384e5 into beam-community:master Dec 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants