Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for Google Wallet passes #501

Merged
merged 2 commits into from
Aug 14, 2024
Merged

Add support for Google Wallet passes #501

merged 2 commits into from
Aug 14, 2024

Conversation

dsluijk
Copy link
Member

@dsluijk dsluijk commented Jul 28, 2024

This PR adds support for Google Wallet, since I am not in Apple's walled garden. It also adds the Apple Wallet button in two more places (confirmation email and the payment success screen).

Do not merge until the Wallet account is reviewed and approved.

@robertdijk
Copy link
Member

The passes don't work for anonymous checkouts, since you have to be a users, and the owner of the ticket when you want a google/apple wallet pass. Maybe we should hide the buttons if it was an anonymous checkout?

@JoepdeJong
Copy link
Contributor

does this mean data is shared with Google?

@dsluijk
Copy link
Member Author

dsluijk commented Jul 28, 2024

The passes don't work for anonymous checkouts

This should work, but I will check. These passes aren't really tied to a user, since it's just a fancy QR-code display.

does this mean data is shared with Google?

This is only done when a user clicks on the button, and only for that user. So I believe that is fine.

@dsluijk dsluijk marked this pull request as ready for review July 28, 2024 11:31
@dsluijk dsluijk changed the title DRAFT: Add support for Google Wallet passes Add support for Google Wallet passes Jul 28, 2024
@dsluijk dsluijk enabled auto-merge July 29, 2024 20:18
@dsluijk dsluijk merged commit a74eed0 into master Aug 14, 2024
5 checks passed
@dsluijk dsluijk deleted the feat/googlepasses branch August 14, 2024 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants