Skip to content

#276 Venmo #281

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

#276 Venmo #281

wants to merge 1 commit into from

Conversation

yurg
Copy link

@yurg yurg commented Mar 26, 2025

No description provided.

@civibot civibot bot added the master label Mar 26, 2025
@yurg yurg mentioned this pull request Mar 26, 2025
@eileenmcnaughton
Copy link
Owner

@yurg does this have any impact on existing users we should understand?

@yurg
Copy link
Author

yurg commented Mar 27, 2025

@eileenmcnaughton Hard to tell - in my case there has been only one CiviCRM instance having Venmo issues, which this PR has resolved. I don't have any other Venmo-enabled setups.

@eileenmcnaughton
Copy link
Owner

@yurg ok - but have you used this patch on non-venmo enabled setups - I want to check they won't break

@yurg
Copy link
Author

yurg commented Mar 28, 2025

@eileenmcnaughton No, I have not. There is one more project having the omnipay extension installed and activated, but it uses a different payment gateway. I can apply the patch there if it helps.

@eileenmcnaughton
Copy link
Owner

@yurg I guess if they are not using paypal it would not affect them?

@yurg
Copy link
Author

yurg commented Mar 28, 2025

@eileenmcnaughton That's right.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants