Skip to content

Commit

Permalink
Merge branch 'supporter_level_goal' into 4207-add-first-name-merge-ta…
Browse files Browse the repository at this point in the history
…g-for-use-in-mailchimp-journey
  • Loading branch information
wwahammy authored Oct 1, 2023
2 parents 5130b50 + 489c5e8 commit 6d2fae2
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion app/views/mailchimp/nonprofit_user_subscribe.json.jbuilder
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,6 @@ json.email_address @user.email
json.status 'subscribed'

json.merge_fields do
json.NONPROFIT_ID @nonprofit.id
json.NP_ID @nonprofit.id
json.FNAME @user.calculated_first_name || ""
end
2 changes: 1 addition & 1 deletion spec/lib/mailchimp_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,7 @@
'email_address' => user.email,
'status' => 'subscribed',
'merge_fields' => {
'NONPROFIT_ID' => nonprofit.id,
'NP_ID' => nonprofit.id,
'FNAME' => "",
}
})
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,7 @@
email_address: User.first.email,
status: 'subscribed',
merge_fields: {
NONPROFIT_ID: User.first.roles.first.host.id,
NP_ID: User.first.roles.first.host.id,
FNAME: ""
}
)
Expand All @@ -43,7 +43,7 @@
email_address: User.first.email,
status: 'subscribed',
merge_fields: {
NONPROFIT_ID: User.first.roles.first.host.id,
NP_ID: User.first.roles.first.host.id,
FNAME: User.first.name
}
)
Expand Down

0 comments on commit 6d2fae2

Please sign in to comment.