-
Notifications
You must be signed in to change notification settings - Fork 265
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ui: adding alerts for robot account actions (PROJQUAY-5946) #2228
Conversation
Codecov ReportAll modified lines are covered by tests ✅ see 2 files with indirect coverage changes 📢 Thoughts on this report? Let us know!. |
ac35785
to
a155057
Compare
...es/OrganizationsList/Organization/Tabs/DefaultPermissions/createTeamWizard/AddTeamMember.tsx
Outdated
Show resolved
Hide resolved
web/src/hooks/useRobotAccounts.ts
Outdated
const queryClient = useQueryClient(); | ||
|
||
const createRobotAccntMutator = useMutation( | ||
const updateRobotData = async (result) => { |
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.
These calls aren't handled with error handling. A success will still be reported to the user if the robot was created successfully even if these all fail.
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.
Yes, I think this was done to make sure a robotAcct is created before adding repos/teams/etc to the robot acct.
26ede52
to
91800cb
Compare
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.
LGTM
No description provided.