Skip to content

Fix dead receipt proof and share actions - #1427

Merged
GoodnessJohn merged 2 commits into
Haroldwonder:mainfrom
Quanwritescodes:fix/sr-180-181-receipt-actions
Aug 30, 2026
Merged

Fix dead receipt proof and share actions#1427
GoodnessJohn merged 2 commits into
Haroldwonder:mainfrom
Quanwritescodes:fix/sr-180-181-receipt-actions

Conversation

@Quanwritescodes

Copy link
Copy Markdown

Summary

This PR fixes two dead actions in the transaction receipt flow:

  • The proof-of-payout link now opens the payout URL through the native Linking API and shows a visible alert when the URL is unsupported or cannot be opened.
  • The share button now calls the native React Native share sheet instead of only opening a read-only modal preview.

What changed

  • Wired the proof-of-payout link to Linking.openURL with URL validation.
  • Added a failure path that surfaces an Alert.alert instead of failing silently.
  • Replaced the misleading modal-only share flow with a real Share.share invocation that includes a plain-text receipt summary.
  • Added focused screen tests covering successful share behavior and proof-of-payout open/failure paths.

Notes

  • This is intentionally scoped to the two receipt actions requested by SR-180 and SR-181.
  • No broad test suite was run as part of this surgical fix.

Checklist

  • Fix dead proof-of-payout action
  • Fix dead share receipt action
  • Add targeted screen tests
  • Keep PR scope limited to the receipt issue"

closes #1323
closes #1324

@vercel

vercel Bot commented Aug 30, 2026

Copy link
Copy Markdown

@darcszn is attempting to deploy a commit to the Harold's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Aug 30, 2026

Copy link
Copy Markdown

@Quanwritescodes Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@GoodnessJohn
GoodnessJohn merged commit 1d5c1e7 into Haroldwonder:main Aug 30, 2026
1 check failed
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.

SR-181: Make 'Share receipt' actually invoke native sharing SR-180: Wire up the dead 'Proof of payout' button in the receipt view

3 participants