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.

Sign up for GitHub

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

Jump to bottom

[REF] [Towards membership api] more consistent handling on amount #19911

Merged
merged 1 commit into from
Apr 1, 2021
Merged

[REF] [Towards membership api] more consistent handling on amount #19911

merged 1 commit into from
Apr 1, 2021

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

[REF] [Towards membership api] more consistent handling on amount

Before

Parameter passed around

After

Parameter retrieved

Technical Details

This also moves template assignments to the email function

Comments

@eileenmcnaughton
[REF] [Towards membership api] more consistent handling on amount
53480c5 
This also moves template assignments to the email function
@civibot civibot bot added the master label Mar 26, 2021
@civibot
Copy link

civibot bot commented Mar 26, 2021

(Standard links)

@eileenmcnaughton
Copy link
Contributor Author

@monishdeb and this one

@monishdeb
Copy link
Member

r-run passed . Patch looks good.

@monishdeb monishdeb merged commit 788b304 into civicrm:master Apr 1, 2021
@eileenmcnaughton eileenmcnaughton deleted the new branch April 1, 2021 03:11
@eileenmcnaughton
Copy link
Contributor Author

thanks @monishdeb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
master
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@eileenmcnaughton @monishdeb