integrallis / stripe_event

Stripe webhook integration for Rails applications.
https://rubygems.org/gems/stripe_event
MIT License
840 stars 107 forks source link

Add support fot stripe v9 #162

Closed taise closed 3 months ago

taise commented 10 months ago

Closes #160

Add support for stripe v9

taise commented 9 months ago

Hi @rmm5t,

I'd appreciate it if you review this pull request when you have a moment. Thanks!

mikker commented 9 months ago

I'm running this in production and can confirm that everything works fine.

abepark01 commented 8 months ago

@taise thank you! It looks like Stripe recently updated their ruby gem to v10.0.0.

seanharmer commented 7 months ago

Ping, could we get this merged please? And then also move on to stripe version 10?

oboxodo commented 7 months ago

I've just pushed a follow-up PR built on top of this, adding support for stripe 10: #163.

AxelTheGerman commented 6 months ago

It also looked to me that there would be no breaking changes that would affect this gem in general (only however folks chose to implement their events)... would be great to get this merged soon

mikker commented 6 months ago

fwiw I got tired f waiting and replaced this gem with a homegrown setup. Didn't take much. Here's basically all there is to it if you want to do the same:

https://gist.github.com/mikker/9405c9dc473d07ddfeefec95b38284c0

AxelTheGerman commented 6 months ago

fwiw I got tired f waiting and replaced this gem with a homegrown setup. Didn't take much. Here's basically all there is to it if you want to do the same:

https://gist.github.com/mikker/9405c9dc473d07ddfeefec95b38284c0

Thanks for sharing, that's a shame because the whole point of gems like this is to reduce the little bit of boilerplate needed there.

Not that I'm expecting lots of development, this is literally feature complete - but without any attention in 4 months this is not passing the bar :disappointed:

AxelTheGerman commented 6 months ago

@rmm5t @peterkeen @invisiblefunnel is this something you could still look at or does the community need to move on?