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

PROD-33: Add civicrm_contribution_recur to LineItem supported entity options #404

Merged
merged 1 commit into from
Nov 25, 2021

Conversation

olayiwola-compucorp
Copy link
Contributor

Overview

CiviCRM decided to limit the supported entity_table options to some predefined tables as seen in this PR civicrm/civicrm-core#20464, this is now causing error in this extension, because this extension allow users to add LineItems to civicrm_contribution_recur table which is not part of the predefined tables.

Before

343d1bc5-732a-4a6c-bdd7-fb3cee8f9b01

After

after-line-item

Technical Details

implement the hook_civicrm_fieldOptions hook to add civicrm_contribution_recur to the LineItem options.

@omarabuhussein
Copy link
Member

omarabuhussein commented Oct 26, 2021

@olayiwola-compucorp ah interesting, so we don't need this anymore compucorp/civicrm-core#78 or even a PR to the core right ?

@olayiwola-compucorp
Copy link
Contributor Author

@olayiwola-compucorp ah interesting, so we don't need this anymore compucorp/civicrm-core#78 or even a PR to the core right ?

Yes, we don't need that anymore

@omarabuhussein omarabuhussein force-pushed the PROD-33-fix-add-lineitem branch from e4dfe85 to f8a434b Compare November 25, 2021 21:02
@omarabuhussein
Copy link
Member

Given that this passed testing already, and that we will need it for V5, I rebased it with the current master branch (which includes V5 now) and will merge it.

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.

2 participants