Skip to content

fix: skip Content-Type on body-less requests and cover wallet rejections (#228, #233, #234) - #349

Merged
presidojay1 merged 1 commit into
StellarTickets:mainfrom
devwura:fix/wallet-errors-and-api-request-hardening
Oct 20, 2026
Merged

presidojay1 merged 1 commit into
StellarTickets:mainfrom
devwura:fix/wallet-errors-and-api-request-hardening

Conversation

@devwura

@devwura devwura commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Closes #228
Closes #233
Closes #234
Closes #229

Testing

  • npx eslint . passes
  • npx tsc --noEmit passes
  • npm run build succeeds
  • npm test passes

These checks were not run for this PR. The new tests are in src/lib/api-fetch.test.ts, src/app/marketplace/page.test.tsx and src/app/dashboard/events/[id]/page.test.tsx. Please run them in CI or locally.

A body-less GET still forced a CORS preflight because Content-Type was always set. Set the header only when a body is sent (StellarTickets#228). Add tests showing the Freighter rejection message on the marketplace buy (StellarTickets#234) and event publish (StellarTickets#233) actions.
@devwura
devwura requested a review from presidojay1 as a code owner October 20, 2026 23:43
@drips-wave

drips-wave Bot commented Oct 20, 2026

Copy link
Copy Markdown

@devwura 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

@netlify

netlify Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

❌ Deploy Preview for stellartickets failed.

Name Link
🔨 Latest commit 3fdb0d5
🔍 Latest deploy log https://app.netlify.com/projects/stellartickets/deploys/6ad7fca373d84326fd6d1df0

@presidojay1
presidojay1 merged commit 5269129 into StellarTickets:main Oct 20, 2026
0 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants