Add a confirm delete dropdown to delete overrides

[harshsbhat]

Preliminary Checks

• I have reviewed https://unkey.com/docs for existing features that would solve my problem

• I have searched for existing feature requests: https://github.com/unkeyed/unkey/issues

• This issue is not a question, general help request, or anything other than a feature request directly related to Unkey. Please ask questions in our Discord community: https://unkey.com/discord.

Is your feature request related to a problem? Please describe.

Currently, if you try to delete overrides. It is immediately deleted. This might not be ideal as someone might miss-click it and have their override gone within a second.

https://app.unkey.com/ratelimits/namespaceId/overrides/

Describe the solution

Add a confirm modal like every other delete button to ask the user whether they want to delete their override before actually deleting it.

Describe alternatives you have considered (if any)

No response

Additional context

No response

[linear]

ENG-1396 Add a confirm delete modal to delete overrides

[chronark]

I'm not sure here, it's not a critical piece, and deleting it on accident doesn't have many consequences.

[harshsbhat]

Yeah, maybe not a written modal like others. But a click-based confirm modal should do right?

[pragyananda]

i am working on it .

[harshsbhat]

i am working on it .

its not approved

[pragyananda]

ok, no problem if this is approved then please assign me.

[chronark]

Yeah, maybe not a written modal like others. But a click-based confirm modal should do right?

yeah, I was thinking this too
maybe turning the button into a dropdown with a single "confirm delete" action

[harshsbhat]

Yeah, maybe not a written modal like others. But a click-based confirm modal should do right?

yeah, I was thinking this too maybe turning the button into a dropdown with a single "confirm delete" action

Sounds good to me. Can you remove the needs approval tag? @pragyananda wants to give a shot most likely

[Aryant01]

can you assign me this issue to me

[biswa2005]

/assign

[oss-gg]

Assigned to @biswa2005! Please open a draft PR linking this issue within 48h ⚠️ If we can't detect a PR from you linking this issue in 48h, you'll be unassigned automatically 🕹️ Excited to have you ship this 🚀

[harshsbhat]

@chronark don't want to sound like this. But this should award me 50-150 points right?

[chronark]

/award 150

[oss-gg]

Awarding harshsbhat: 150 points 🕹️ Well done! Check out your new contribution on oss.gg/harshsbhat

[ShreyasLakhani]

/assign

[oss-gg]

This issue is already assigned to another person. Please find more issues here.

[roshan-29]

/assign

[oss-gg]

This issue is already assigned to another person. Please find more issues here.

[Naresh-official]

/assign

[oss-gg]

This issue is already assigned to another person. Please find more issues here.

[Navnit73]

/assign

[oss-gg]

This issue is already assigned to another person. Please find more issues here.