Skip to content

Memoize role list#7106

Merged
nischalstha9 merged 1 commit intodevelopfrom
fix/7100-updates-all-with-same-roles-follow-up-fix-eslint
Dec 5, 2025
Merged

Memoize role list#7106
nischalstha9 merged 1 commit intodevelopfrom
fix/7100-updates-all-with-same-roles-follow-up-fix-eslint

Conversation

@suzit-10
Copy link
Copy Markdown
Contributor

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

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

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

Related Issue

Example: Fixes #7100

Describe this PR

Memoized the role list and fixed the ESLint warning.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Dec 5, 2025

@suzit-10 suzit-10 changed the title Filter out roles and show only remaining roles in team options Memoize role list Dec 5, 2025
@suzit-10 suzit-10 requested a review from nischalstha9 December 5, 2025 07:49
@nischalstha9 nischalstha9 merged commit cc4ed75 into develop Dec 5, 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

2 participants