-
Notifications
You must be signed in to change notification settings - Fork 20
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
feat(osp): endpoint to get onboarding osp companies details #866
feat(osp): endpoint to get onboarding osp companies details #866
Conversation
...s/administration/Administration.Service.Tests/BusinessLogic/RegistrationBusinessLogicTest.cs
Fixed
Show fixed
Hide fixed
...s/administration/Administration.Service.Tests/BusinessLogic/RegistrationBusinessLogicTest.cs
Fixed
Show fixed
Hide fixed
...s/administration/Administration.Service.Tests/BusinessLogic/RegistrationBusinessLogicTest.cs
Fixed
Show fixed
Hide fixed
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.
#808 is flagged with milestone 2.1.0 (24.08), if that one is still valid directing the pull request to main is wrong, and it should go to the release branch, could you please clarify?
sure will directing to release branch |
@AnuragNagpure please rebase to release/v2.1.0-RC2 and make sure you synched to the latest version before. it seems like there are 8 commits in that shouldn't be in |
e7e5f8a
to
26d3f35
Compare
26d3f35
to
0c576f6
Compare
I just removed the unwanted commits by doing an interactive rebase |
c66e151
to
4c1cde0
Compare
I did fix all of my review-findings |
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.
testing for 24.08 ends officially in less than 24h, please don't merge without me confirming but the change will very likely not go into the 2.1.0 release anymore, I'll let you know tomorrow
confirmed: v.2.1.0 is closed, please change to main |
The base branch was changed.
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.
One minor finding
application.Company.CompanyAssignedRoles.Select(companyAssignedRoles => companyAssignedRoles.CompanyRoleId), | ||
application.Company.IdentityProviders.Select(x => new IdentityProvidersDetails(x.Id, x.IamIdentityProvider!.IamIdpAlias)), | ||
application.Company.BusinessPartnerNumber, | ||
application.Company.Identities.Where(x => x.CompanyUser!.Identity!.UserStatusId != UserStatusId.DELETED).Count())) |
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.
This can be changed to
application.Company.Identities.Where(x => x.CompanyUser!.Identity!.UserStatusId != UserStatusId.DELETED).Count())) | |
application.Company.Identities.Count(x => x.CompanyUser!.Identity!.UserStatusId != UserStatusId.DELETED))) |
21106a6
to
919203f
Compare
Quality Gate passedIssues Measures |
@evegufy can you please approve that we can merge. |
Description
Why
Need this api to retrieve a list of all Onboarded OSP (Onboarding Service Provider) companies along with their current ### status
Issue
#808
Checklist
Please delete options that are not relevant.