fix: Reading role permissions generates 500 errors

[novakzaballa]

Thanks for submitting a PR! Please check the boxes below:

• I have run pre-commit to check linting
• I have filled in the "Changes" section below?
• I have filled in the "How did you test this code" section below?
• I have used a Conventional Commit title for this Pull Request

Changes

• Skip requests when the role organization is not sent
• Use new RBAC endpoint to retrieve and delete userRoles by user

How did you test this code?

• Go to manage -> members -> Users
• Open the Network tab
• Click any User
• The error is no longer there

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
docs	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 8, 2024 7:05pm
flagsmith-frontend-preview	🛑 Canceled (Inspect)			Jan 8, 2024 7:05pm
flagsmith-frontend-staging	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Jan 8, 2024 7:05pm

[github-actions]

Uffizzi Preview deployment-41151 was deleted.

[matthewelwell]

Based on this Slack thread, I'm not sure this is the right solution. All roles should have an organisation attribute. I've marked this PR as draft until we have understood more about this.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (9f83f27) 95.99% compared to head (b54b420) 95.99%.
Report is 2 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #3009   +/-   ##
=======================================
  Coverage   95.99%   95.99%           
=======================================
  Files        1062     1062           
  Lines       32297    32297           
=======================================
  Hits        31005    31005           
  Misses       1292     1292           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[gagantrivedi]

Looks like it's trying to use environment API key for organisation ID?

[novakzaballa]

Looks like it's trying to use environment API key for organisation ID?

Corrected.

[gnumoreno]

@novakzaballa I'm not sure if this code touches on it but if I go to https://app.flagsmith.com/organisation-settings -> member - groups - create a new group, the new group does not show up. I have to navigate to complete refresh the page for the group to show up.

[gnumoreno]

Other than the group thing I commented above, the approvers seems to be working just fine. I still think the user experience is confusing but I will take some time later this week and open a separate issue

[novakzaballa]

@novakzaballa I'm not sure if this code touches on it but if I go to https://app.flagsmith.com/organisation-settings -> member - groups - create a new group, the new group does not show up. I have to navigate to complete refresh the page for the group to show up.

@gnumoreno That part of the code was not touched, this is a good catch, we would have to create an issue.