Remove AcceptOrgUserByOrgId from AcceptOrgUserCommand to simplify interface

[lizard-boy]

Type of change

- [ ] Bug fix
- [ ] New feature development
- [X] Tech debt (refactoring, code cleanup, dependency upgrades, etc)
- [ ] Build/deploy pipeline (DevOps)
- [ ] Other

Objective

In researching a bug, I identified that there was the potential to remove one of the methods by which we accept a user into an org. We could simplify the interface and also avoid re-retrieving the OrganizationUser when we already have it in the calling method.

Code changes

• file.ext: Description of what was changed and why

Before you submit

• Please check for formatting errors (dotnet format --verify-no-changes) (required)
• If making database changes - make sure you also update Entity Framework queries and/or migrations
• Please add unit tests where it makes sense to do so (encouraged but not required)
• If this change requires a documentation update - notify the documentation team
• If this change has particular deployment requirements - notify the DevOps team

Greptile Summary

This pull request removes the AcceptOrgUserByOrgId method from the AcceptOrgUserCommand interface and updates related classes to simplify the codebase and avoid redundant data retrieval.

• Removed AcceptOrgUserByOrgIdAsync method from src/Core/AdminConsole/OrganizationFeatures/OrganizationUsers/Interfaces/IAcceptOrgUserCommand.cs
• Updated src/Api/AdminConsole/Controllers/OrganizationUsersController.cs to use AcceptOrgUserAsync with orgUser object directly
• Modified test/Api.Test/AdminConsole/Controllers/OrganizationUsersControllerTests.cs to reflect new method signature
• Deleted corresponding test cases in test/Core.Test/AdminConsole/OrganizationFeatures/OrganizationUsers/AcceptOrgUserCommandTests.cs

[greptile-apps]

LGTM

_{5 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile}