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

COMCL-887: Support renewal of paid contribution #541

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

Conversation

olayiwola-compucorp
Copy link
Contributor

@olayiwola-compucorp olayiwola-compucorp commented Oct 24, 2024

Overview

Before this PR membership with paid contributions did not get renewed, this is because of a recurring contribution condition that filters out recurring contributions with non-empty end date

Before

Membership did not get renewed, and no new contribution was created after running the renewal job
Screenshot 2024-10-24 at 14 02 59

After

Running the renewal job now renews the membership with a completed contribution and creates a new pending contribution
Screenshot 2024-10-24 at 13 59 46

Technical Details

This issue doesn't happen in the previous CiviCRM version because completing a contribution does not change the end of a recurring contribution

@jamienovick
Copy link

@olayiwola-compucorp could you explain the technical details in a bit more detail please? I'm a little confused as to what changed in 3.x -> 4.x in order for this to become a problem? Doe the fact that completing the contribution "doesn't change the end of a recurring contribution" have other knock on effects?

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

Successfully merging this pull request may close these issues.

3 participants