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

Add utm_* parameters to Facebook and Twitter shares #130

Open
wwahammy opened this issue Jan 4, 2019 · 4 comments
Open

Add utm_* parameters to Facebook and Twitter shares #130

wwahammy opened this issue Jan 4, 2019 · 4 comments
Labels
good first issue Good for newcomers

Comments

@wwahammy
Copy link
Member

wwahammy commented Jan 4, 2019

Currently, the Facebook and Twitter share buttons in the followup pane of the donation wizard include a link to the donation page. That donation page link does not include any utm_* parameter. For Houdini instances which use analytics on their donation/campaign/event pages, it would be useful for those shares to include the utm_* parameters so people can better track which donations came from which source.

IMPORTANT PRIVACY NOTE: we should avoid including utm_* parameter values which are so specific that we can track inbound traffic by individual user.

@wwahammy wwahammy added the good first issue Good for newcomers label Jan 4, 2019
@Philwi
Copy link

Philwi commented Jul 7, 2022

Hey, should the utm_parameters be configurable for the users or should they be generated by the given data. Like the actual campaign etc.? Should the parameters always be sent or just if an utm-configuration is toggled?
There are these utm-paramenters: https://en.wikipedia.org/wiki/UTM_parameters

I could imagine that the source parameter is auto-generated but the rest of it is configured by the user.

Am I right that there are no integration tests but the tests for the react components?

If there would be some guidance I would start with this ticket. :)

@victordscabral
Copy link
Contributor

victordscabral commented Dec 13, 2022

Hi! Since there is no recent update in this issue me (@victordscabral), @kayrocesar and @joaobisi will work on it.

@victordscabral
Copy link
Contributor

@wwahammy. A doubt. Is this the file to add the utm parameters?
image

@victordscabral
Copy link
Contributor

Link to the PR: #1473

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

3 participants