Closed tmock12 closed 9 years ago
Looks good to me! I will pull this onboard but will wait until I have time to rewrite the tutorial.
@DanielKehoe any word on when you plan to update the tutorial?
@cayblood thanks for your interest. I'm currently writing the tutorials for rails-devise and rails-devise-pundit. Then I will update the rails-prelaunch-signup app before undertaking an update for the rails-stripe-membership-saas app. Each example app and tutorial takes about six weeks to produce. So it will be several months before I update this one.
There is a new version of the application for Rails 4.2.
I extracted the stripe methods to their own class. This reduced the method sizes and made them easier to test. It also reduced the large "update_stripe" method in the user model. That thing had a lot of complexity in it. There's still some more that could probably be extracted but this seems to be a good start? thoughts?