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

SDK-4541 Add organization support for client credentials to node-auth0 #1033

Merged
merged 2 commits into from
Sep 9, 2024

Conversation

tusharpandey13
Copy link
Contributor

@tusharpandey13 tusharpandey13 commented Sep 1, 2024

Changes

This PR includes changes for the feature 'Add organization support for client credentials to node-auth0'.
Changed managers:

  • clients-manager
  • organizations-manager
  • resource-servers-manager
    and respective tests and models.
client credentials code changes SDK-4541
Client creds tests updated
added missing fields related to SS changes
added missing models related to client update
fix linting issue

References

Testing

Unit tests added for changed endpoints.

  • This change adds unit test coverage
  • This change adds integration test coverage

Checklist

@tusharpandey13 tusharpandey13 requested a review from a team as a code owner September 1, 2024 17:56
@gyaneshgouraw-okta
Copy link
Contributor

Hey @tusharpandey13, will it be possible to add public api links instead of internal reference links for the supported api's

@tusharpandey13
Copy link
Contributor Author

Hey @tusharpandey13, will it be possible to add public api links instead of internal reference links for the supported api's

Added in the comments, please note that GET /api/v2/organizations/{org_id}/client-grants path is not yet present in public docs.

nandan-bhat
nandan-bhat previously approved these changes Sep 3, 2024
@tusharpandey13 tusharpandey13 merged commit b9eb2d0 into master Sep 9, 2024
11 checks passed
@tusharpandey13 tusharpandey13 deleted the feature/SDK-4541 branch September 9, 2024 06:50
@tusharpandey13 tusharpandey13 mentioned this pull request Sep 10, 2024
5 tasks
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.

3 participants