Skip to content

Use skinny Google Wallet JWT (lost in #187 merge conflict)#191

Merged
denbec merged 1 commit into
mainfrom
denbec/wallet-google-skinny-jwt
Jun 8, 2026
Merged

Use skinny Google Wallet JWT (lost in #187 merge conflict)#191
denbec merged 1 commit into
mainfrom
denbec/wallet-google-skinny-jwt

Conversation

@denbec

@denbec denbec commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator

The skinny-JWT change from #187 was dropped during merge-conflict resolution. main still inlines the EventTicketClass with reviewStatus: UNDER_REVIEW on every save JWT, which can restrict saves to the issuer + test accounts.

This references the existing approved class programmiercon_ticket_v4 by id and sends only the EventTicketObject — the exact version verified to save with a normal Google account.

Note: the actual production blocker (a non-numeric GOOGLE_WALLET_ISSUER_ID) was already fixed in the deployed .env (correct numeric issuer id 3388000000023114464). This restores the code-side cleanup so production stops sending the under-review class inline.

🤖 Generated with Claude Code

The skinny-JWT change from PR #187 was dropped during the merge-conflict
resolution: main still inlines the EventTicketClass with
`reviewStatus: UNDER_REVIEW` in every save JWT. Re-declaring the
already-APPROVED class as under review can make Google treat the save as a
test/preview (only the issuer + allowlisted test accounts can save).

Reference the existing approved class by id and send only the
EventTicketObject, matching the version that was verified to save with a
normal Google account.

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

vercel Bot commented Jun 8, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
programmierbar-website Building Building Preview, Comment Jun 8, 2026 4:44pm

Request Review

@denbec denbec merged commit 76fda0f into main Jun 8, 2026
2 of 3 checks passed
@denbec denbec deleted the denbec/wallet-google-skinny-jwt branch June 8, 2026 16:44
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