Skip to content

725-create_editTeamsPermissions: Created a new endpoint (#725)#127

Closed
Subhadeepghosh1 wants to merge 3 commits intodevfrom
725-create_editTeamsPermissions
Closed

725-create_editTeamsPermissions: Created a new endpoint (#725)#127
Subhadeepghosh1 wants to merge 3 commits intodevfrom
725-create_editTeamsPermissions

Conversation

@Subhadeepghosh1
Copy link
Copy Markdown
Contributor

Description:
Created a new endpoint(/editGroupDetails)

Changes Made:
AdminGroupAuthorizationResource.java

@Subhadeepghosh1 Subhadeepghosh1 requested a review from AAfghahi May 9, 2025 13:53
@Subhadeepghosh1 Subhadeepghosh1 self-assigned this May 9, 2025
@Subhadeepghosh1 Subhadeepghosh1 requested review from anurag91jain and removed request for AAfghahi May 9, 2025 13:55
@Subhadeepghosh1
Copy link
Copy Markdown
Contributor Author

This pr should be merged along with SEMOSS/Semoss#803

@AAfghahi
Copy link
Copy Markdown
Contributor

Fix the error and we'll get this reviewed.

@Subhadeepghosh1
Copy link
Copy Markdown
Contributor Author

Subhadeepghosh1 commented May 13, 2025

@AAfghahi
The build is failing because the endpoint is calling a method editGroupDetailsAndPropagate, which is not present in the current branch. This method is included in the following pr: SEMOSS/Semoss#803
So, to resolve the build issue, the Semoss PR need to be reviewed and merged first so that the method can be accessible here.

@kunal0137
Copy link
Copy Markdown
Contributor

how do i test this? where on the ui is the call being made to this ?

@kunal0137
Copy link
Copy Markdown
Contributor

why is this code here twice? #130

@anurag91jain
Copy link
Copy Markdown

Changes were merged as part of another PR.

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.

[TASK] Create editTeamsPermissions reactor

5 participants