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

Fixes support subscription cancelation #3636

Merged
merged 1 commit into from
Oct 26, 2023

Conversation

eliduke
Copy link
Collaborator

@eliduke eliduke commented Oct 25, 2023

The stripe gem deprecated subscription delete. The new way is now cancel.

What does this pull request do?

This should fix the issue with canceling subscriptions, like this...

Screenshot 2023-10-25 at 10 37 12 AM

How should this be manually tested?

Yes, I will try canceling my subscription once this goes live.

The stripe gem deprecated subscription `delete`. The new way is now `cancel`. This should fix the issue with canceling subscriptions.
@just1602 just1602 merged commit fcb49ae into main Oct 26, 2023
7 checks passed
@just1602 just1602 deleted the fixes-support-subscription-cancelation branch October 26, 2023 03:20
@just1602
Copy link
Collaborator

@eliduke I merged it, but I don't have the access to promote to production, so @veganstraightedge will have to do it.

@eliduke
Copy link
Collaborator Author

eliduke commented Dec 25, 2023

btw, I finally got around to testing this and confirmed that it is indeed working! I was able to cancel my subscription. The only problem now is that I'm realizing that I'm not seeing any flash notices appear at any point. We've got a bunch of them in place in the support controller for a number of things. Upon further inspection, I'm seeing a bunch of different syntax like...

flash[:notice]
flash.now[:notice]
flash.now.alert

Are flash notices still working? Did the syntax get updated? I'm seeing some stuff regarding "themes" (2017 or 2020) and it seems like the flash notice partial isn't available in 2020.

Thoughts?

@just1602
Copy link
Collaborator

@eliduke rubocop enforce flash.new[:flash_type] syntax, but both are supposed to work.

Otherwise, you should check in the 2017 theme, the 2020 theme never got fully implemented due to covid and people having less time for the project.

@bensheldon
Copy link
Contributor

hopefully helpful: ☺️

flash.now should be used when the message is to be shown when the current request's template is rendered i.e. it's not a redirect.

I think that flash[:type] likely shouldn't be used, and flash messages instead should be added to the redirect_to the_path, flash: { type: "message" }

(And yes, the theme should be double checked that they are rendered)

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 this pull request may close these issues.

3 participants