-
Notifications
You must be signed in to change notification settings - Fork 41
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
extensionworkshop.com links without UTM params #8254
Comments
Yes, let's add some UTM params. :) Note: these are just the params -- let me know if they need reformatting per #8252. olympia • src\olympia\devhub\templates\devhub\agreement.html: olympia • src\olympia\lib\crypto\tasks.py: olympia • src\olympia\devhub\templates\devhub\emails\submission.txt: olympia • src\olympia\devhub\templates\devhub\addons\submit\describe_minimal.html: "The source code must meet policy requirements, which includes:" should have the parameters: olympia • src\olympia\devhub\templates\devhub\addons\submit\describe.html: olympia • src\olympia\devhub\templates\devhub\addons\submit\distribute.html: olympia • src\olympia\devhub\templates\devhub\emails\submission.html: Resources for publishers: ?utm_source=email&utm_medium=submission-email olympia • src\olympia\devhub\templates\devhub\new-landing\components\banner.html: About tracking external sources (side note, just want to call out how meta it feels to be adding a UTM parameter to a link that describes UTM parameters...) |
It's referenced in an email we are/were planning to send when add-ons are bulk-resigned with a new signature |
Got it. We don't need to add UTM params to that. |
@caitmuenster I would like to contribute to this issue. I think the below mentioned ones are meant for
Also, what UTM params need to be added to links in |
Hi @kanish671, I am so sorry for the late response! Yes, you've got the right idea about the UTM params for I don't need anything for |
I'm closing this issue because I'm not sure what pieces have been done and what still needs to be fixed. 🙃 I'll open new issues if/when we find UTMs that need to be added. |
From searching for #8252 I realised we have a number of links to extensionworkshop.com that don't have any UTM param
@caitmuenster should they be added to all of these? And if so, can you provide them?
Note the mix of
settings.EXTENSION_WORKSHOP_URL
and the literal url. IMO we should update all the links to use the setting if we're taking the time to make a change.The text was updated successfully, but these errors were encountered: