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

CRM-19850 - Add a offline membership without 'record payment' option,… #11087

Merged
merged 1 commit into from
Mar 23, 2018

Conversation

yashodha
Copy link
Contributor

@yashodha yashodha commented Oct 9, 2017

… later edit the membership and saving with 'record payment' option

Overview

  1. Visit any contact, and click on membership tab.
  2. Click on 'Add Membership', select the desired membership, and uncheck 'record payment' option, save it.
  3. Now, edit the same membership, and this time check the 'record payment' checkbox, the payment details form doesn't appear.
  4. Trying to save this screen, results in 'Please select Financial Type' form error message.

Before

membership_contri_edit

… later edit the membership and saving with 'record payment' option
@yashodha
Copy link
Contributor Author

yashodha commented Oct 9, 2017

@jitendrapurohit can you please look at this?

@yashodha yashodha added the master label Oct 9, 2017
@eileenmcnaughton
Copy link
Contributor

I tested this & replicated the bug and was able to see that this fixes the bug. The code change is small and targetted & I feel confident it won't affect other things.

I think this is a short term fix, as we move to replace this with the payment block, but definitely a fix worth having, merging

@eileenmcnaughton eileenmcnaughton merged commit c1151e8 into civicrm:master Mar 23, 2018
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