Skip to content

Role duplicatioin on update#7101

Merged
ramyaragupathy merged 1 commit intodevelopfrom
fix/7100-updates-all-with-same-roles
Dec 4, 2025
Merged

Role duplicatioin on update#7101
ramyaragupathy merged 1 commit intodevelopfrom
fix/7100-updates-all-with-same-roles

Conversation

@suzit-10
Copy link
Copy Markdown
Contributor

@suzit-10 suzit-10 commented Dec 3, 2025

What type of PR is this? (check all applicable)

  • 🍕 Feature
  • 🐛 Bug Fix
  • 📝 Documentation
  • 🧑‍💻 Refactor
  • ✅ Test
  • 🤖 Build or CI
  • ❓ Other (please specify)

Related Issue

Describe this PR

Previously, when updating a role, the logic filtered the role list only by team ID. As a result, updating a role would apply the selected role to all roles under that team.
Now, the update logic also checks the current role in addition to the team ID, ensuring that only the specifically selected role is identified and updated.

…h selection

This ensures only the selected role is updated
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Dec 3, 2025

@suzit-10 suzit-10 changed the title Update role by ensuring both team id and current role are matched wit… Role duplicatioin on update Dec 3, 2025
@manjitapandey manjitapandey self-requested a review December 3, 2025 12:08
@ramyaragupathy ramyaragupathy merged commit c6259ba into develop Dec 4, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

If the same team is assigned for multiple roles, editing one permission updates all with the same role in edit project page

3 participants