fix(frontend): Legal terms hint names DEX wallets, not Keplr-only (#658) #1157

Merged
PlasticDigits merged 2 commits from feat/658-legal-wallet-hint into main 2026-08-26 08:09:22 +00:00
PlasticDigits commented 2026-08-26 06:54:44 +00:00 (Migrated from gitlab.com)

Summary

  • Retarget WC-M12 / C1: after connect without a keplr-like signer injector, the Legal next-step hint names the DEX Connect set (Station, Keplr, Cosmostation, Lunc Dash, Galaxy Station) or the connected wallet — not Keplr-only.
  • Hide the hint when window.keplr, window.station.keplr, or window.cosmostation.providers.keplr is present (hasLegalSignerInjector reuses getKeplrLikeExtension). 'station' in window without the shim still shows the hint.
  • Accept Terms still navigates to the Legal portal. DEX still does not implement ADR-036. Leap is not named.

Closes #658

Test plan

  • make verify-issue-658 (hint + gate + clickwrap unit tests, L658/WC-M12/C1 greps)
  • make verify-issue-554 (updated WC-M12 assertions; not Keplr-only)
  • make verify-issue-517
  • make verify-issue-566
  • Manual columbus-5 / https://dex.cl8y.com unsigned connect (desktop Keplr/Station/Cosmostation extension; Android WC; in-app browsers) per issue test plan
## Summary - Retarget **WC-M12** / **C1**: after connect without a keplr-like signer injector, the Legal next-step hint names the DEX Connect set (Station, Keplr, Cosmostation, Lunc Dash, Galaxy Station) or the connected wallet — not Keplr-only. - Hide the hint when `window.keplr`, `window.station.keplr`, or `window.cosmostation.providers.keplr` is present (`hasLegalSignerInjector` reuses `getKeplrLikeExtension`). `'station' in window` without the shim still shows the hint. - Accept Terms still navigates to the Legal portal. DEX still does not implement ADR-036. Leap is not named. Closes #658 ## Test plan - [x] `make verify-issue-658` (hint + gate + clickwrap unit tests, L658/WC-M12/C1 greps) - [x] `make verify-issue-554` (updated WC-M12 assertions; not Keplr-only) - [x] `make verify-issue-517` - [x] `make verify-issue-566` - [ ] Manual columbus-5 / `https://dex.cl8y.com` unsigned connect (desktop Keplr/Station/Cosmostation extension; Android WC; in-app browsers) per issue test plan
PlasticDigits commented 2026-08-26 08:08:56 +00:00 (Migrated from gitlab.com)

added 6 commits

  • 494c066f...dc869a7f - 5 commits from branch main
  • 723c4268 - Merge origin/main into feat/658-legal-wallet-hint

Compare with previous version

added 6 commits <ul><li>494c066f...dc869a7f - 5 commits from branch <code>main</code></li><li>723c4268 - Merge origin/main into feat/658-legal-wallet-hint</li></ul> [Compare with previous version](/PlasticDigits/cl8y-dex-terraclassic/-/merge_requests/439/diffs?diff_id=1984419901&start_sha=494c066f01413f97ec995332c8f16305bbc40bb0)
PlasticDigits commented 2026-08-26 08:09:23 +00:00 (Migrated from gitlab.com)

mentioned in commit c2a6592da5

mentioned in commit c2a6592da5cc84ac3fcfbf1d36a6dad0df4d2d09
PlasticDigits (Migrated from gitlab.com) merged commit c2a6592da5 into main 2026-08-26 08:09:23 +00:00
PlasticDigits commented 2026-08-26 09:30:51 +00:00 (Migrated from gitlab.com)

mentioned in issue #658

mentioned in issue #658
PlasticDigits commented 2026-08-26 09:31:36 +00:00 (Migrated from gitlab.com)

mentioned in issue #673

mentioned in issue #673
Sign in to join this conversation.
No reviewers
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
code/cl8y-dex-terraclassic!1157
No description provided.