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-20375: Show message instead of checkbox when renew required. Hide checkbox w… #10095

Merged

Conversation

mattwire
Copy link
Contributor

@mattwire mattwire commented Apr 3, 2017

…hen renew not allowed. On confirmation page change message depending if user selected auto-renew or it was mandatory

…hen renew not allowed. On confirmation page change message depending if user selected auto-renew or it was mandatory
Copy link
Contributor

@jitendrapurohit jitendrapurohit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've tested the display message on main/confirm/thankyou page which seems to be working correctly. I think this is good to merge.

@eileenmcnaughton
Copy link
Contributor

Thanks @mattwire & @jitendrapurohit - merging

@eileenmcnaughton eileenmcnaughton merged commit d37f296 into civicrm:master Apr 5, 2017
@mattwire mattwire deleted the membership_force_auto_renew branch April 5, 2017 23:01
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