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
Branch
deleteContact2
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 18854
Build 37496: Build Diffcashtab-tests
Build 37495: arc lint + arc unit

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.