Company form: remove an identifier with a bin and a confirmation, not a checkbox #857

Closed
opened 2026-10-10 16:01:27 +00:00 by tiagoagueda · 1 comment
Owner

Replace the Remove checkbox on a company's identifier rows with a bin icon that removes at once after a confirmation dialog.

What exists today

  • templates/cotton/identifier_row.html:63-66: the formset's DELETE box, applied on Save.
  • Your details already does it the other way: remove_at_once (:61-62) draws <c-remove-button :removal="row.removal"> (trash-2, popovertarget) and a <c-dialog action=… live data-remove-row> outside the form (profile.html:307, :336-349; accounts/removals.py).

What would change

  • A removal endpoint for CompanyIdentifier, row.removal set in the company view, and the dialogs drawn after </form>.

Note #759 (PST-030): the formset discloses whether another account's identifier id exists; a removal endpoint must check ownership the same way.

Checked against main at 8c3d03570. From the maintainer's improvements list of 2026-10-07.

Replace the *Remove* checkbox on a company's identifier rows with a bin icon that removes at once after a confirmation dialog. ## What exists today - `templates/cotton/identifier_row.html:63-66`: the formset's `DELETE` box, applied on Save. - Your details already does it the other way: `remove_at_once` (`:61-62`) draws `<c-remove-button :removal="row.removal">` (`trash-2`, `popovertarget`) and a `<c-dialog action=… live data-remove-row>` outside the form (`profile.html:307`, `:336-349`; `accounts/removals.py`). ## What would change - A removal endpoint for `CompanyIdentifier`, `row.removal` set in the company view, and the dialogs drawn after `</form>`. Note #759 (PST-030): the formset discloses whether another account's identifier id exists; a removal endpoint must check ownership the same way. Checked against `main` at `8c3d03570`. From the maintainer's improvements list of 2026-10-07.
tiagoagueda added this to the 0.6.0 milestone 2026-10-10 16:01:27 +00:00
Author
Owner

Fixed in ef8679cd3 on main.

Fixed in ef8679cd3 on main.
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
Postulo/postulo#857
No description provided.