Skip to content

Fix Share on X not preloading tweet on mobile#90

Merged
techgangboss merged 1 commit intomainfrom
fix-share-x-mobile
Mar 31, 2026
Merged

Fix Share on X not preloading tweet on mobile#90
techgangboss merged 1 commit intomainfrom
fix-share-x-mobile

Conversation

@techgangboss
Copy link
Copy Markdown
Contributor

Summary

  • Switch from twitter.com/intent/tweet to x.com/intent/post (modern endpoint)
  • Separate text and url params instead of embedding URL in text body
  • Fixes mobile deep link handoff dropping query params — X app opens but composer is empty

The old twitter.com/intent/tweet endpoint doesn't reliably pass ?text= through the mobile deep link → X app bridge. The x.com/intent/post endpoint handles this correctly. Separating &url= also ensures the link card renders even if text gets truncated.

Note: the old comment said &url= caused Twitter composer to get stuck reloading. That was a twitter.com domain issue — x.com/intent/post handles it correctly.

Test plan

  • Mobile: tap Share on X after preconfirmation — X app should open with tweet draft pre-filled
  • Mobile: tap Share on X after "Tokens Available" — same behavior
  • Desktop: Share on X opens x.com composer with text + link card
  • Verify OG card still renders from the share URL

🤖 Generated with Claude Code

Switch from twitter.com/intent/tweet to x.com/intent/post — the legacy
endpoint drops query params during mobile deep link handoff to the X app.
Also separate text and url params so the link card renders independently.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 31, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
fastprotocolapp Ready Ready Preview, Comment Mar 31, 2026 5:21am

Request Review

@techgangboss techgangboss merged commit 6fe2824 into main Mar 31, 2026
1 of 3 checks passed
@techgangboss techgangboss deleted the fix-share-x-mobile branch March 31, 2026 05:18
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