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 payment_status in default doPayment per dev/financial#141 #20021

Merged
merged 1 commit into from
Apr 10, 2021

Conversation

mattwire
Copy link
Contributor

@mattwire mattwire commented Apr 9, 2021

Overview

Step towards https://lab.civicrm.org/dev/financial/-/issues/141 by returning payment_status as well as payment_status_id

Before

Core doPayment does not return payment_status.

After

Core doPayment returns payment_status.

Technical Details

Comments

@eileenmcnaughton @artfulrobot

@civibot
Copy link

civibot bot commented Apr 9, 2021

(Standard links)

@civibot civibot bot added the master label Apr 9, 2021
@eileenmcnaughton
Copy link
Contributor

Thanks @mattwire - this looks completely in line with what was agreed

@eileenmcnaughton eileenmcnaughton merged commit 149e111 into civicrm:master Apr 10, 2021
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