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-20901, Added submit credit card membership link #10689

Merged
merged 1 commit into from
Jul 18, 2017

Conversation

pradpnayak
Copy link
Contributor


----------------------------------------
* CRM-20901: Add submit credit card membership link on membership form
  https://issues.civicrm.org/jira/browse/CRM-20901
@pradpnayak pradpnayak changed the title CRM-20901, Added submit credit card membership link [ready for core team to review]CRM-20901, Added submit credit card membership link Jul 18, 2017
@eileenmcnaughton
Copy link
Contributor

@jitendrapurohit can you test this - looks fine / safe to me

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.

Tested.

@monishdeb
Copy link
Member

@eileenmcnaughton @colemanw can you please merge this PR as approved by Jitendra?

@colemanw
Copy link
Member

Makes sense

@colemanw colemanw merged commit b4407c5 into civicrm:master Jul 18, 2017
@monishdeb monishdeb deleted the CRM-20901 branch July 18, 2017 13:14
@eileenmcnaughton eileenmcnaughton changed the title [ready for core team to review]CRM-20901, Added submit credit card membership link CRM-20901, Added submit credit card membership link Jul 19, 2017
@eileenmcnaughton
Copy link
Contributor

@monishdeb @pradpnayak @JoeMurray can we only use the negative [WIP] in the titles - if there is no WIP then a PR should be ready for review by any reviewer who steps up. It's pretty ugly to have the [ready for core team review] in our it history & the fact it's ready for review should be implicit (unless {WIP} is in the title

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.

6 participants