Skip to content
This repository has been archived by the owner on Jan 5, 2023. It is now read-only.

Confirmation option to delete resource #549

Open
ghost opened this issue Jan 11, 2021 · 9 comments
Open

Confirmation option to delete resource #549

ghost opened this issue Jan 11, 2021 · 9 comments
Assignees

Comments

@ghost
Copy link

ghost commented Jan 11, 2021

As a data publisher, when I am managing the resources on a dataset, I want to be asked to confirm I want to delete a resource, so that I don't accidentally delete things.

Notes:

  • Let's find an existing modal pattern to implement
@ghost ghost assigned adborden Jan 11, 2021
@adborden
Copy link
Contributor

I would not recommend using a third-party framework. I would just borrow from existing USWDS examples:

@ghost ghost assigned anuveyatsu Jan 25, 2021
@ghost
Copy link
Author

ghost commented Jan 25, 2021

@anuveyatsu @osahon-okungbowa Can we get an estimate on this?

@osahon-okungbowa
Copy link

@cmugisha @adborden this ticket will be estimated. Please confirm if @anuveyatsu can begin working on this immediately after estimating

@ghost
Copy link
Author

ghost commented Jan 26, 2021

@osahon-okungbowa Can he work on this and on #2654 simultaneously? It doesn't look like #2654 was estimated...

@osahon-okungbowa
Copy link

@cmugisha #2654 has been completed and deployed to sandbox . I left a comment on it

@ghost
Copy link
Author

ghost commented Jan 26, 2021

Great! Then Anu can start working on this after he estimates it. Thanks!

@ghost
Copy link
Author

ghost commented Jan 26, 2021

@osahon-okungbowa I just saw Aaron's comment in Slack about #554 being higher priority, so let me confirm that in scrum and then get back to you. Sorry for the confusion.

@osahon-okungbowa
Copy link

@cmugisha moved this to QA. Ticket is being deployed to sandbox

@ghost ghost assigned hkdctol and unassigned adborden and anuveyatsu Feb 10, 2021
@hkdctol
Copy link
Contributor

hkdctol commented Feb 12, 2021

Confirmed on sandbox--hitting delete on existing resource resulted in box asking to confirm.

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants