[Cashtab] Do not use alert modal for agora.cash
ClosedPublic
Actions

Authored by bytesofman on Jun 5 2025, 21:30.

Details

Reviewers

emack

Group Reviewers

Restricted Project

Commits

rABCd88a3b987f12: [Cashtab] Do not use alert modal for agora.cash

Summary

User asked about this in the main tg. This link is ok.

Test Plan

npm test

Diff Detail

Repository

rABC Bitcoin ABC

Branch

moar-safe-links

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 33553
Build 66585: Build Diff	cashtab-tests
Build 66584: arc lint + arc unit

Event Timeline

bytesofman created this revision.Jun 5 2025, 21:30

Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 5 2025, 21:30

bytesofman published this revision for review.Jun 5 2025, 21:30

Harbormaster completed remote builds in B33553: Diff 54371.Jun 5 2025, 21:34

emack accepted this revision.Jun 6 2025, 02:34

This revision is now accepted and ready to land.Jun 6 2025, 02:34

Closed by commit rABCd88a3b987f12: [Cashtab] Do not use alert modal for agora.cash (authored by bytesofman). · Explain WhyJun 6 2025, 04:03

This revision was automatically updated to reflect the committed changes.

bytesofman added a commit: rABCd88a3b987f12: [Cashtab] Do not use alert modal for agora.cash.

Revision Contents
Changeset List

Path

Size

cashtab/

src/

components/

Common/

UncontrolledLink.tsx

1 line

Diff 54371

View Options

cashtab/src/components/Common/UncontrolledLink.tsx

[Cashtab] Do not use alert modal for agora.cashClosedPublicActions