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.

Sign up for GitHub

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

Jump to bottom

Add API Resource Management API #473

Merged
merged 6 commits into from
Sep 18, 2023
Merged

Add API Resource Management API #473

merged 6 commits into from
Sep 18, 2023

Conversation

ThaminduR
Copy link
Contributor

@ThaminduR ThaminduR commented Aug 9, 2023 •
edited
Loading

Purpose

Add API Resource Management API

Goals

Add the API Resource Management API. Allows managing API Resources in the identity server.

Issue

wso2/product-is#16344

Automation tests

  • Unit tests
  • Integration tests

Related PRs

Depends on wso2/carbon-identity-framework#4854

@ThaminduR ThaminduR marked this pull request as ready for review August 16, 2023 04:39
@ThaminduR ThaminduR mentioned this pull request Aug 16, 2023
Merged
sahandilshan
sahandilshan requested changes Aug 22, 2023
View reviewed changes
Copy link
Contributor

@sahandilshan sahandilshan left a comment

Choose a reason for hiding this comment

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

Error handling needs to be improved

inthirakumaaran
inthirakumaaran reviewed Aug 23, 2023
View reviewed changes
pom.xml Outdated Show resolved Hide resolved
@ThaminduR ThaminduR force-pushed the add-api-resource branch 2 times, most recently from 3b209f1 to 7e64211 Compare September 18, 2023 08:09
sahandilshan
sahandilshan previously approved these changes Sep 18, 2023
View reviewed changes
AnuradhaSK
AnuradhaSK previously approved these changes Sep 18, 2023
View reviewed changes
@ThaminduR ThaminduR merged commit 49cdb21 into wso2:master Sep 18, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sadilchamishka sadilchamishka sadilchamishka left review comments

@asha15 asha15 asha15 left review comments

@ShanChathusanda93 ShanChathusanda93 ShanChathusanda93 left review comments

@inthirakumaaran inthirakumaaran inthirakumaaran left review comments

@thanujalk thanujalk thanujalk left review comments

@madurangasiriwardena madurangasiriwardena madurangasiriwardena left review comments

@jenkins-is-staging jenkins-is-staging jenkins-is-staging approved these changes

@AnuradhaSK AnuradhaSK AnuradhaSK approved these changes

@sahandilshan sahandilshan sahandilshan left review comments

@janakamarasena janakamarasena Awaiting requested review from janakamarasena

@shashimalcse shashimalcse Awaiting requested review from shashimalcse

@SujanSanjula96 SujanSanjula96 Awaiting requested review from SujanSanjula96

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
10 participants
@ThaminduR @jenkins-is-staging @madurangasiriwardena @thanujalk @inthirakumaaran @ShanChathusanda93 @AnuradhaSK @asha15 @sahandilshan @sadilchamishka