Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fixed Management API Grants class instantiation #179

Merged
merged 2 commits into from
Jan 4, 2019
Merged

Fixed Management API Grants class instantiation #179

merged 2 commits into from
Jan 4, 2019

Conversation

beck3905
Copy link
Contributor

@beck3905 beck3905 commented Jan 4, 2019

fixed typo by changing '.' to ','

@beck3905
Copy link
Contributor Author

beck3905 commented Jan 4, 2019

@lbalmaceda - This is a very minor fix to a bug in the latest version that causes the management API not to work. Could you please review and release this as soon as possible? Thanks.

Copy link
Contributor

@lbalmaceda lbalmaceda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@beck3905 nice catch! In order for this not to happen again, would you add some tests to that file as well please? You can copy-paste this file and add the missing methods/classes https://github.com/auth0/auth0-python/pull/164/files#diff-1e4900791205f36d990eff57682110bb

@beck3905
Copy link
Contributor Author

beck3905 commented Jan 4, 2019

@lbalmaceda - Done!

@lbalmaceda lbalmaceda merged commit eb4b017 into auth0:master Jan 4, 2019
@lbalmaceda lbalmaceda added this to the v3-Next milestone Jan 4, 2019
@lbalmaceda lbalmaceda modified the milestones: v3-Next, 3.6.1 Jan 4, 2019
@beck3905
Copy link
Contributor Author

beck3905 commented Jan 4, 2019

@lbalmaceda - Thank you. When will the next release with this fix be available on PyPi?

@lbalmaceda lbalmaceda changed the title fixed typo by changing '.' to ',' Fixed Management API Grants class instantiation Jan 4, 2019
@lbalmaceda
Copy link
Contributor

It's already there 😉 under 3.6.1

@beck3905
Copy link
Contributor Author

beck3905 commented Jan 4, 2019

Just saw it come through. Thanks.

@beck3905 beck3905 deleted the fix-grants-typo branch January 4, 2019 20:00
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants