-
Notifications
You must be signed in to change notification settings - Fork 12
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
Delete button on silo images table #1797
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
ra.resource_type === resourceType && | ||
ra.resource_id === resourceId && | ||
roleOrStronger[role].includes(ra.role_name) | ||
) |
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.
Honestly this was very easy. Not sure why I didn't do this before.
Well.... this does not account for parent-child relationships, like if you have collab on silo you also have it on all projects in silo. I need to either account for that or add a comment about the limitation.
Edit: Comment added in 771f5d7
oxidecomputer/console@3538c32...672cb20 * [672cb208](oxidecomputer/console@672cb208) bump API spec (BGP endpoints, not used yet) * [a50777c5](oxidecomputer/console@a50777c5) oxidecomputer/console#1795 * [8978e07a](oxidecomputer/console@8978e07a) bump omicron version (no changes) * [992fb1e1](oxidecomputer/console@992fb1e1) oxidecomputer/console#1797 * [e5b8d029](oxidecomputer/console@e5b8d029) remove unnecessary @types/testing-library__jest-dom dep * [e9a78617](oxidecomputer/console@e9a78617) tanstack query 5 went stable
oxidecomputer/console@3538c32...8e74acc * [8e74accf](oxidecomputer/console@8e74accf) handle empty metrics list on silo utilization * [672cb208](oxidecomputer/console@672cb208) bump API spec (BGP endpoints, not used yet) * [a50777c5](oxidecomputer/console@a50777c5) oxidecomputer/console#1795 * [8978e07a](oxidecomputer/console@8978e07a) bump omicron version (no changes) * [992fb1e1](oxidecomputer/console@992fb1e1) oxidecomputer/console#1797 * [e5b8d029](oxidecomputer/console@e5b8d029) remove unnecessary @types/testing-library__jest-dom dep * [e9a78617](oxidecomputer/console@e9a78617) tanstack query 5 went stable
oxidecomputer/console@3538c32...bd65b9d * [bd65b9da](oxidecomputer/console@bd65b9da) loading state for utilization table, use QueryParamTabs * [8e74accf](oxidecomputer/console@8e74accf) handle empty metrics list on silo utilization * [672cb208](oxidecomputer/console@672cb208) bump API spec (BGP endpoints, not used yet) * [a50777c5](oxidecomputer/console@a50777c5) oxidecomputer/console#1795 * [8978e07a](oxidecomputer/console@8978e07a) bump omicron version (no changes) * [992fb1e1](oxidecomputer/console@992fb1e1) oxidecomputer/console#1797 * [e5b8d029](oxidecomputer/console@e5b8d029) remove unnecessary @types/testing-library__jest-dom dep * [e9a78617](oxidecomputer/console@e9a78617) tanstack query 5 went stable
oxidecomputer/console@3538c32...bd65b9d * [bd65b9da](oxidecomputer/console@bd65b9da) loading state for utilization table, use QueryParamTabs * [8e74accf](oxidecomputer/console@8e74accf) handle empty metrics list on silo utilization * [672cb208](oxidecomputer/console@672cb208) bump API spec (BGP endpoints, not used yet) * [a50777c5](oxidecomputer/console@a50777c5) oxidecomputer/console#1795 * [8978e07a](oxidecomputer/console@8978e07a) bump omicron version (no changes) * [992fb1e1](oxidecomputer/console@992fb1e1) oxidecomputer/console#1797 * [e5b8d029](oxidecomputer/console@e5b8d029) remove unnecessary @types/testing-library__jest-dom dep * [e9a78617](oxidecomputer/console@e9a78617) tanstack query 5 went stable
Closes #1796
onError
on project image delete to fix double toast (delete confirm handles error toast for us)