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

Wallet Orders #136

Closed
cjnewbs opened this issue Feb 2, 2024 · 1 comment · Fixed by #137
Closed

Wallet Orders #136

cjnewbs opened this issue Feb 2, 2024 · 1 comment · Fixed by #137

Comments

@cjnewbs
Copy link
Contributor

cjnewbs commented Feb 2, 2024

Summary

Not really an issue as such but thought this would be the best place to ask. I've been working on creating Wallet orders and got completely stuck with the signature generation. Thankfully I came across this stackoverflow post which pointed me in this direction and now am able to create files recognised by the wallet app.

It looks like Wallet Passes and Orders are quite similar with only some minor differences.

I was considering trying to create a backwards-compatible PR to upgrade this repo to support generation of orders too. Is this something you would be interested in or would you prefer to focus strictly on passes (in which case I will try create a new library to create distributable order packages)?

If you want to focus only on Passes feel free to close this issue.

@tschoffelen
Copy link
Owner

Hi Craig! Ah that's cool, I didn't realise that's how wallet orders worked! (if anyone else ends up reading this, see here)

We could certainly extend it, as long as we do it in a way that isn't confusing to devs wanting to create passes.

Maybe we should introduce a separate class called PKOrder or something similar for it? We could then create a base class that shares some of the signing logic.

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 a pull request may close this issue.

2 participants