Skip to content

Fix mobile MetaMask deeplink and polish steward review UI#622

Merged
JoaquinBN merged 1 commit into
devfrom
JoaquinBN/mobile-wallet-connect
May 13, 2026
Merged

Fix mobile MetaMask deeplink and polish steward review UI#622
JoaquinBN merged 1 commit into
devfrom
JoaquinBN/mobile-wallet-connect

Conversation

@JoaquinBN
Copy link
Copy Markdown
Collaborator

Summary

  • Drops encodeURIComponent on the MetaMask dapp path and switches to link.metamask.io; mobile MetaMask was opening its in-app browser to a blank tab because the encoded /, ?, # produced a malformed destination URL. Also adds a Coinbase Wallet deep link (go.cb-w.com/dapp?cb_url=...).
  • Renders contribution type descriptions as markdown on the type detail page, preserves line breaks, clamps to six lines by default, and shows a Show more / Show less toggle only when the content actually overflows.
  • Surfaces each user's linked GitHub / X / Discord accounts as compact brand pills next to the user name on the steward submissions view; backend StewardSubmissionSerializer.get_user_details now always merges the three social connections into user_details and the queryset adds matching select_related to avoid N+1.

Test plan

  • On a real phone with MetaMask Mobile installed, tap Connect → MetaMask and verify the in-app browser actually loads the dapp (including hash routes) instead of a blank tab.
  • Open a contribution type detail page whose description contains markdown / multiple paragraphs and confirm it renders formatted, clamps to ~6 lines, and the Show more / Show less toggle works.
  • Visit Stewards → Submissions and confirm GitHub / X / Discord pills appear next to users who have those accounts linked, with the GitHub and X pills linking to their external profile.

Drops encodeURIComponent on the MetaMask dapp path and moves to
link.metamask.io so the in-app browser actually loads the dapp instead
of opening a blank tab; also adds a Coinbase Wallet deep link. Renders
contribution type descriptions as markdown with a clamped-to-six-lines
preview and Show more toggle on the type detail page. Surfaces each
user's linked GitHub/X/Discord accounts next to their name on the
steward submissions view (backend serializer now merges social
connections into user_details with the matching select_related).
@JoaquinBN JoaquinBN merged commit 52e30d1 into dev May 13, 2026
@JoaquinBN JoaquinBN deleted the JoaquinBN/mobile-wallet-connect branch May 13, 2026 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant