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

Make 'Pending' the default status for Contribution.repeattransaction #17432

Merged

Conversation

mattwire
Copy link
Contributor

Overview

As the preferred way to repeat a contribution is now to call Contribution.repeattransaction and then Payment.create it makes sense to set contribution_status_id to Pending by default as that is the preferred way to call this API

Before

Must specify contribution_status_id for repeattransaction.

After

Don't need to specify it. When implementing functionality using Contribution.repeattransaction it is obvious that the intended way to call it is with status=Pending.

Technical Details

Just sets default at API level. As this is a required parameter it won't affect any existing implementations as they will all specify the contribution_status_id as a parameter.

Comments

@eileenmcnaughton

…as that is the preferred way to call this API
@civibot
Copy link

civibot bot commented May 31, 2020

(Standard links)

@civibot civibot bot added the master label May 31, 2020
@eileenmcnaughton
Copy link
Contributor

OK - agreed & it won't break existing as they will pass in a default

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