Skip to content
This repository was archived by the owner on Oct 29, 2020. It is now read-only.

Fixes Issue 1304, missing CAMPAIGN_LINK merge_var in campaign_signup#1316

Merged
deezone merged 1 commit intoDoSomethingArchive:devfrom
deezone:Issue1304-Missing_campaign_link
Mar 17, 2014
Merged

Fixes Issue 1304, missing CAMPAIGN_LINK merge_var in campaign_signup#1316
deezone merged 1 commit intoDoSomethingArchive:devfrom
deezone:Issue1304-Missing_campaign_link

Conversation

@deezone
Copy link
Contributor

@deezone deezone commented Mar 17, 2014

Fixes #1304

  • Adds CAMPAIGN_LINK to Message Broker merge_var payload values for campaign signups.

@aaronschachter
Copy link
Contributor

This looks ok. General note to team bender that we'll want to have to redirect turned on for paths that change, so these email links don't 404 if paths change.

@aaronschachter
Copy link
Contributor

Update: We do have redirect turned on to 301 for paths that change, hooray.

deezone added a commit that referenced this pull request Mar 17, 2014
Fixes Issue 1304, missing CAMPAIGN_LINK merge_var in campaign_signup
@deezone deezone merged commit add495f into DoSomethingArchive:dev Mar 17, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

email - prove it link in email body copy not linked properly

2 participants