Page MenuHomePhabricator
Differential D14832

[Cashtab] Modify unknown msg warning
AbandonedPublic
Actions

Authored by bytesofman on Nov 21 2023, 20:07.

Details

Reviewers
None
Group Reviewers
Restricted Project
Summary

It's actually a bit common for people to receive msgs from outside their contact list for normal reasons, e.g. a token airdrop. The warning is enough without the direct 'scam' beware.

Test Plan

npm test

Diff Detail

Repository
rABC Bitcoin ABC
Branch
remove-scam-warning
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 25697
Build 50974: Build Diffcashtab-tests
Build 50973: arc lint + arc unit

Event Timeline

bytesofman created this revision.Nov 21 2023, 20:07
Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 21 2023, 20:07
bytesofman requested review of this revision.Nov 21 2023, 20:07
Harbormaster completed remote builds in B25697: Diff 43205.Nov 21 2023, 20:09
Fabien added a subscriber: Fabien.Nov 21 2023, 21:38

What is the problem with the actual warning ? It feels more secure to me.

bytesofman abandoned this revision.Nov 21 2023, 21:46
In D14832#333157, @Fabien wrote:

What is the problem with the actual warning ? It feels more secure to me.

Response to this user complaint which, on reflection, is probably a common reaction:

https://twitter.com/wenfeng1168/status/1727046070533013739

...on consideration tho... this is probably a better reaction than the alternative. Would be nice to have a more specific msg but that isn't practical without overcomplicating things.

Revision Contents
Changeset List

PathSize
cashtab/
src/
components/
Home/
3 lines

Diff 43205

View Options

cashtab/src/components/Home/Tx.js

Loading...