Page MenuHomePhabricator

[Cashtab] Delete contacts
ClosedPublic

Authored by emack on Apr 13 2022, 01:58.

Details

Reviewers
bytesofman
Group Reviewers
Restricted Project
Commits
rABC1d1f4a9d7a46: [Cashtab] Delete contacts
Summary

Supersedes D11324.

Contact deletion function.

Test Plan

Verify the Delete contact icon is not rendered in an empty contact list
Add a contact and ensure deletion via the confirmation modal is successful

Diff Detail

Repository
rABC Bitcoin ABC
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

emack requested review of this revision.Apr 13 2022, 01:58

inline comment + rebase to latest master

web/cashtab/src/components/Configure/Configure.js
832 ↗(On Diff #33183)

Let's make this the contact name instead of the last 4 chars of the contact.

This revision now requires changes to proceed.Apr 20 2022, 23:17
emack marked an inline comment as done.

Adjusted deletion confirmation to contact name and rebased to master

This revision is now accepted and ready to land.Apr 21 2022, 17:21
This revision was automatically updated to reflect the committed changes.