- Notifications
You must be signed in to change notification settings - Fork927
fix: use insert and delete instead of upsert for custom roles#14252
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
fix: use insert and delete instead of upsert for custom roles#14252
Uh oh!
There was an error while loading.Please reload this page.
Conversation
14da05a toc567a9dCompareffa67cf to7042119Compare| if (role) { | ||
| await updateOrganizationRoleMutation.mutateAsync(data); | ||
| navigate(`/organizations/${organizationName}/roles`); | ||
| } else { | ||
| await createOrganizationRoleMutation.mutateAsync(data); | ||
| navigate(`/organizations/${organizationName}/roles`); | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
| if(role){ | |
| awaitupdateOrganizationRoleMutation.mutateAsync(data); | |
| navigate(`/organizations/${organizationName}/roles`); | |
| }else{ | |
| awaitcreateOrganizationRoleMutation.mutateAsync(data); | |
| navigate(`/organizations/${organizationName}/roles`); | |
| } | |
| if(role){ | |
| awaitupdateOrganizationRoleMutation.mutateAsync(data); | |
| }else{ | |
| awaitcreateOrganizationRoleMutation.mutateAsync(data); | |
| } | |
| navigate(`/organizations/${organizationName}/roles`); |
alwaysmeticulousbot commentedAug 13, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
🤖 Meticulous replayed 100 user sessions andtook 1355 visual snapshots. Meticulous has not yet run onc567a9d of the main branch and so there was nothing to compare against. Last updated for commit161ee26. This comment will update as new commits are pushed. |
05f1814 intostevenmasley/custom_role_remove_upsertUh oh!
There was an error while loading.Please reload this page.
resolves#14245