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 the capability to update discovery configuration of the primary organization. #410

Merged
merged 4 commits into from
Nov 29, 2024

Conversation

Yasasr1
Copy link
Contributor

@Yasasr1 Yasasr1 commented Nov 1, 2024

Purpose

This pr introduces a new method for updating the discovery configuration in the organization configuration management service.

Related issue

Copy link

codecov bot commented Nov 1, 2024

Codecov Report

Attention: Patch coverage is 55.55556% with 8 lines in your changes missing coverage. Please review.

Project coverage is 30.07%. Comparing base (4ad6969) to head (856276d).
Report is 59 commits behind head on main.

Files with missing lines Patch % Lines
.../config/service/OrganizationConfigManagerImpl.java 57.14% 4 Missing and 2 partials ⚠️
...tion/config/service/OrganizationConfigManager.java 0.00% 2 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (4ad6969) and HEAD (856276d). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (4ad6969) HEAD (856276d)
1 0
Additional details and impacted files
@@              Coverage Diff              @@
##               main     #410       +/-   ##
=============================================
- Coverage     41.53%   30.07%   -11.47%     
- Complexity      218      424      +206     
=============================================
  Files            30       61       +31     
  Lines          1678     4323     +2645     
  Branches        187      489      +302     
=============================================
+ Hits            697     1300      +603     
- Misses          938     2913     +1975     
- Partials         43      110       +67     
Flag Coverage Δ
unit 30.07% <55.55%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Yasasr1 Yasasr1 force-pushed the add-self-signup-config branch from c89973d to f507e94 Compare November 28, 2024 03:13
@Yasasr1 Yasasr1 force-pushed the add-self-signup-config branch from f507e94 to 91ad341 Compare November 28, 2024 03:15
@Yasasr1 Yasasr1 merged commit 0974412 into wso2-extensions:main Nov 29, 2024
3 of 4 checks passed
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