-
Notifications
You must be signed in to change notification settings - Fork 41
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
Frontend: New modal for Instance group deletion #767
Frontend: New modal for Instance group deletion #767
Conversation
Hi @cannarelladev. Thanks for your PR. I am @kingmakerbot.
Make sure this PR appears in the CrownLabs changelog, adding one of the following labels:
|
501d433
to
2171e08
Compare
2171e08
to
182adaa
Compare
32ea7a1
to
eae4396
Compare
fbca5cf
to
59ae7ee
Compare
59ae7ee
to
e58c0c9
Compare
/deploy-staging |
/deploy-staging |
2f2a735
to
c248bac
Compare
c248bac
to
5762ee0
Compare
/deploy-staging |
frontend/src/components/activePage/TableTemplate/TableTemplateRow.tsx
Outdated
Show resolved
Hide resolved
5762ee0
to
291e66c
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.
some random suggestions
frontend/src/components/workspaces/Templates/TemplatesTableLogic/TemplatesTableLogic.tsx
Outdated
Show resolved
Hide resolved
frontend/src/components/workspaces/Templates/TemplatesTableLogic/TemplatesTableLogic.tsx
Outdated
Show resolved
Hide resolved
frontend/src/components/activePage/ActiveViewLogic/ActiveViewLogic.tsx
Outdated
Show resolved
Hide resolved
291e66c
to
55749d6
Compare
/deploy-staging |
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.
Mostly stylistic issues, one comment is more concerning though
frontend/src/components/activePage/TableInstance/TableInstance.tsx
Outdated
Show resolved
Hide resolved
frontend/src/components/activePage/TableInstance/TableInstance.tsx
Outdated
Show resolved
Hide resolved
frontend/src/components/activePage/TableTemplate/TableTemplateRow.tsx
Outdated
Show resolved
Hide resolved
frontend/src/components/activePage/TableInstance/TableInstance.tsx
Outdated
Show resolved
Hide resolved
frontend/src/components/activePage/TableInstance/TableInstance.tsx
Outdated
Show resolved
Hide resolved
frontend/src/components/activePage/TableInstance/TableInstance.tsx
Outdated
Show resolved
Hide resolved
frontend/src/components/activePage/TableInstance/TableInstance.tsx
Outdated
Show resolved
Hide resolved
frontend/src/components/activePage/TableInstance/TableInstance.tsx
Outdated
Show resolved
Hide resolved
Co-authored-by: Francesco Cheinasso <cheinasso.francesco@gmail.com>
55749d6
to
e63895a
Compare
/deploy-staging |
/merge |
Your staging environment has been correctly teared-down! |
With this PR will be implement and improve the system and the modal to delete a certain group of instances: