rad-inbox-clear: Add confirmation dialog — heartwood

Some interactive elements (branch dropdowns, the user menu, the clone popover) need JavaScript. Reading repositories, issues, patches, and the docs works without it; submitting forms also works as long as you can reach them via direct links.

heartwood

Radicle Heartwood Protocol & Stack

rad-inbox-clear: Add confirmation dialog

✗ 0/1 checks passed did:key:z6MkoohH...AjJf wants to merge 1 commit into master · opened 2 months ago

This patch adds a confirmation dialog to the potentially-destructive rad inbox clear command.

The default for this confirmation-dialog is set to "Y" for now, but I am of course open to change it to "N" by default - which would IMHO be a better idea for a potentially-destructive operation.

I opted for "Y" for now for better acceptance ;-)

✗ Some checks failed — 0 passed, 1 failed View logs ↗

Activity Commits 1 Changes +47 -17

did:key:z6MkoohH...AjJf opened with revision 1cad7042ba20fec6d37896d206ca1a0a1df6d2ea on base e9245b630d728672d2b3d6ff9265fa4f1a86f13a +47 -17 2 months ago

did:key:z6MkireR...3voM pushed revision 2 6ea9f20b2fad1223ed6aa8e4df7b9fab81eaa1cc on base e9245b630d728672d2b3d6ff9265fa4f1a86f13a +47 -17 2 months ago

REVIEW: fix typo

did:key:z6MkireR...3voM commented on revision 1 2 months ago

I've added a revision to fix a type, see 6ea9f20b2fad1223ed6aa8e4df7b9fab81eaa1cc

Could you also add a CHANGELOG entry?

did:key:z6MkrxqT...yQRh pushed revision 3 4dee5e5996aafe04b4175dd5328f87efc9200a02 on base 5aaf978f972054dd81777d26b47f967eb9d6f00f +48 -17 2 months ago

Incooporated review, added changelog entry, rebased to latest master.