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

Add generic payment method #68

Merged
merged 2 commits into from
Apr 5, 2024
Merged

Add generic payment method #68

merged 2 commits into from
Apr 5, 2024

Conversation

mariav0
Copy link
Contributor

@mariav0 mariav0 commented Jun 5, 2023

adding generic payment method

@MarcMichalsky
Copy link
Contributor

I've created a PR for your branch, @mariav0.

Perhaps it would be good to add an upgrade method that sets a default value for the new payment instrument in each profile. The value would be set anyway the next time you save a profile after the upgrade, but if you forget to do this, a submission with a generic payment will still cause the "Payment method could not be matched to existing payment" error.

@jensschuppe jensschuppe linked an issue Jul 12, 2023 that may be closed by this pull request
@jensschuppe jensschuppe added enhancement New feature or request status:needs work There is code, but it needs additional work before it should be reviewed labels Apr 4, 2024
@jensschuppe jensschuppe changed the title Update Profile.php Add generic payment method Apr 4, 2024
@jensschuppe jensschuppe added this to the 1.5 milestone Apr 4, 2024
@jensschuppe jensschuppe merged commit 6f555c6 into systopia:master Apr 5, 2024
0 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request status:needs work There is code, but it needs additional work before it should be reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

generic payment method not mapped yet
3 participants