Enforce id prop for SingleRoleSelector

[tinahollygb]

Features and Changes

Fixes a bug with using multiple SingleRoleSelector components.

Dependencies

n/a

Testing

• Go to invite a team member: http://localhost:3000/settings/team
• Choose multiple project roles.
• For each of these roles, choose engineer (or another with more privileges)
• Toggle "Restrict Access to Specific Environments" for each of these.

Screenshots

[bryce-fitzsimons]

lgtm

[github-actions]

Your preview environment pr-1072-bttf has been deployed.

Preview environment endpoints are available at:

• https://growthbook-3000-pr-1072-bttf.growthbook.okteto.dev
• https://growthbook-3100-pr-1072-bttf.growthbook.okteto.dev

[bryce-fitzsimons]

lgtm. I might make id an optional prop on SingleRoleSelector, but autogeneration is fine too.

[tinahollygb]

lgtm. I might make id an optional prop on SingleRoleSelector, but autogeneration is fine too.

Thanks @bryce-fitzsimons – yeah we went back and forth on the id prop here but I think the preference is auto-generation for ease of implementation.