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.

Sign up for GitHub

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

Jump to bottom

Sessions not implemented? #120

Open
NorfairKing opened this issue Aug 29, 2019 · 5 comments
Open

Sessions not implemented? #120

NorfairKing opened this issue Aug 29, 2019 · 5 comments

Comments

@NorfairKing
Copy link

This seems to be required for all payments , before September 15 2019. Is there any plan to add this? How can I help?
@dmjio
Copy link
Owner

dmjio commented Aug 30, 2019

At the time this API was implemented, I don't believe sessions were present.

https://web.archive.org/web/20140625062328/https://stripe.com/docs/api

@NorfairKing
Copy link
Author

NorfairKing commented Aug 30, 2019 •
edited
Loading

Yes, that makes sense, but on Sept 15 2019, new regulation comes into effect that requires everyone using stripe to use the new API. This includes sessions.
How can I help to make this happen?

Also, in EventData, the UnknownEventData constructor should probably contain a Data.Aeson.Value.

@NorfairKing
Copy link
Author

The EventType constructor called UnknownEventType should probably also include a Text.

@danstn
Copy link

danstn commented Sep 9, 2019

+1

Currently parsing "checkout.session.completed" event manually. Would be great to have Session support :)

@NorfairKing
Copy link
Author

@danstn Could you help with #122 then?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@danstn @dmjio @NorfairKing