capo (4a7cbb93) at 23 Aug 15:11
A merge request (!166 (merged)) seems to solve the issue by adding setCheckPermissions(FALSE)
to the two API4 calls involved.
Closes #331
A call to the importsubscription
API wasn't completed successfully due to an authorization failed issue:
cv api stripe.importsubscription subscription=sub_XXXXXXX ppid=1 contact_id=1234
{
"error_code": "unauthorized",
"entity": "Stripe",
"action": "importcharge",
"is_error": 1,
"error_message": "Authorization failed"
}
capo (4a7cbb93) at 10 Aug 12:46
Fix stripe.importcharge returns authorization failed
... and 394 more commits
Hi @mattwire,
Your proposal sounds interesting as a roadmap for testing in CiviCRM but I'd suggest to create a new issue to address those suggestions and, for the moment, implement what Rebecca described in the head of the issue. Actually, I have a WIP branch (see my show-test-event-registrations branch) which can solve this issue and it's only missing one or two tests.
Here's the corresponding merge request: https://github.com/civicrm/civicrm-core/pull/20938.
capo (76477f52) at 24 Jul 21:58
capo (07b06ee2) at 24 Jul 21:53
Hi @mattwire,
Can you please take a look at !1 (merged) and let me know if it looks correct to you? It fixes the issue that I describe above, which only occurs if you are not using the \Civi::settings()->get('variablerecurpayments_normalmembershipamount')
setting.
Closes #1
If a recurring contribution is edited in CiviCRM and the amount is written without decimals (like in 30 rather than 30.00):
CiviCRM will show a message saying that the amount was changed to 30.00 and it will store it properly locally:
but in SmartDebit, the amount is saved divided by 100:
At that point, as expected, if the recurring contribution is "viewed", its amount will be updated in CiviCRM to 0.30.
As I said, the issue is actually in the variablerecurpayments. So I'm closing this issue in favor of variablerecurpayments#1 (closed).
If a recurring contribution is edited in CiviCRM and the amount is written without decimals (like in 30 rather than 30.00):
CiviCRM will show a message saying that the amount was changed to 30.00 and it will store it properly locally:
but in SmartDebit, the amount is saved divided by 100:
At that point, as expected, if the recurring contribution is "viewed", its amount will be updated in CiviCRM to 0.30.
capo (76477f52) at 23 Jul 13:25
Fix SmartDebit amounts are divided by 100
Quick note to warn that this may actually be caused by the variablerecurpayments rather than by the smartdebit extension. I'll be adding more information soon.
@mattwire Thanks for the clue. Can you please take a quick look at !7 (merged) and let me know if it looks good to you?
capo (07b06ee2) at 23 Jul 12:26
Fill the organization name token on contribution pages (before subm...
Closes #10