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

Add Guardian Enrollments endpoints #278

Merged
merged 4 commits into from
Aug 8, 2018
Merged

Conversation

fmedinac
Copy link
Contributor

@fmedinac fmedinac commented Jun 26, 2018

This PR adds two new methods on management:

  • getGuardianEnrollment for getting a single guardian enrollment
  • deleteGuardianEnrollment for deleting a guardian enrollment

Note that this PR doesn't include other methods for all the endpoints described on https://auth0.com/docs/api/management/v2#!/Guardian that might be included in future PRs in order to close #249.

Closes #168 (already closed)

Update (July 3, 2018)

Added a new endpoint for regenerating the recovery code (ref: https://auth0.com/docs/api/management/v2#!/Users/post_recovery_code_regeneration)

@cocojoe
Copy link
Member

cocojoe commented Jul 10, 2018

Thanks for this, can you rebase against mater please. I'll ensure this is reviewed. @luisrudge

@fmedinac
Copy link
Contributor Author

@cocojoe done it and it seems our branches are in sync. Please let me know if you need anything.

@cocojoe
Copy link
Member

cocojoe commented Jul 10, 2018

screen shot 2018-07-10 at 14 05 25

I can update for you but prefer people to rebase.

@fmedinac
Copy link
Contributor Author

@cocojoe @luisrudge do you want something else from me that could help it to get reviewed?

@cocojoe
Copy link
Member

cocojoe commented Jul 23, 2018

@fmedinac can you rebase please

@rafaelrinaldi
Copy link

@cocojoe PR seem to be synced with upstream. Do you need anything else in order to get this merged in? Would love to have it available.

@luisrudge
Copy link
Contributor

I'll review this tomorrow.

@rafaelrinaldi
Copy link

@luisrudge Thank you. If you need any help to get this in feel free to ping me. I can help.

@@ -61,6 +61,10 @@ Auth0RestClient.prototype.patch = function(/* [params], [callback] */) {
return this.wrappedProvider('patch', arguments);
};

Auth0RestClient.prototype.post = function(/* [params], [callback] */) {
return this.wrappedProvider('post', arguments);
Copy link
Contributor

Choose a reason for hiding this comment

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

we don't need 'post'. you have to use 'create', which is already being used through out the code base.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@luisrudge
Copy link
Contributor

Hi @rafaelrinaldi and @fmedinac. I left just a single comment. Can you write a quick script file that uses all the new methods so I can test that they're correctly communicating with the API?
You can use brucke.auth0.com as the domain and TnlRxGUgmqOYWt8pkZTlXcttyzRoLm7q as clientID.

@fmedinac
Copy link
Contributor Author

fmedinac commented Aug 1, 2018

@luisrudge I've created a repository for testing it. See if it helps: https://github.com/fmedinac/node-auth0-278-poc

Please see if it works for you.

@rafaelrinaldi
Copy link

@luisrudge No rush, just want to check and see how things are looking. We can make any changes necessary so this can get merged, just let us know.

@luisrudge
Copy link
Contributor

Sorry. Took me a while to get to this! Thank you so much for the repo, it was super easy to test the changes!

@luisrudge luisrudge merged commit a990594 into auth0:master Aug 8, 2018
@luisrudge luisrudge added this to the v2.12.0 milestone Aug 9, 2018
@rafaelrinaldi
Copy link

@luisrudge Any idea as of when this change is gonna get published as a new release?

@luisrudge
Copy link
Contributor

Hi, we just published 2.12.0. Sorry about the wait!

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

Successfully merging this pull request may close these issues.

How to delete guardian multifactor?
4 participants