-
Notifications
You must be signed in to change notification settings - Fork 374
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
Allow OAuth2 application management through DCR in sub organizations #2578
Allow OAuth2 application management through DCR in sub organizations #2578
Conversation
92e8d33
to
725f30e
Compare
725f30e
to
5fb251d
Compare
5fb251d
to
4a4332e
Compare
f7ec0ad
to
dd68cd0
Compare
...identity.oauth.dcr/src/main/java/org/wso2/carbon/identity/oauth/dcr/service/DCRMService.java
Outdated
Show resolved
Hide resolved
ea2d920
to
8ddbb51
Compare
Please take one of the following actions:
Your prompt attention to this matter is greatly appreciated. Thank you for your understanding and collaboration! 🙏 |
This PR is in WIP and will be completed in next couple of days. Hence keeping this open as for now. |
a670f83
to
5637863
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #2578 +/- ##
============================================
- Coverage 56.40% 56.39% -0.02%
- Complexity 8269 8273 +4
============================================
Files 632 632
Lines 46680 46815 +135
Branches 8508 8520 +12
============================================
+ Hits 26332 26402 +70
- Misses 16585 16644 +59
- Partials 3763 3769 +6
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
5637863
to
a6843bf
Compare
...ntity.oauth.dcr/src/main/java/org/wso2/carbon/identity/oauth/dcr/internal/DCRDataHolder.java
Show resolved
Hide resolved
...arbon.identity.oauth/src/main/java/org/wso2/carbon/identity/oauth/OAuthAdminServiceImpl.java
Show resolved
Hide resolved
PR builder started |
PR builder completed |
a6843bf
to
ef64efd
Compare
6afd2b7
ef64efd
to
6afd2b7
Compare
PR builder started |
PR builder completed |
6afd2b7
to
f1420ad
Compare
PR builder started |
PR builder completed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving the pull request based on the successful pr build https://github.com/wso2/product-is/actions/runs/12633558767
Proposed changes in this pull request
/t/{tenant-domain}/o/{org-id}/api/identity/oauth2/dcr/v1.1/register
When should this PR be merged