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

Feature/utm hypercharge #7502

Merged
merged 9 commits into from
Oct 4, 2020

Conversation

zoek1
Copy link
Contributor

@zoek1 zoek1 commented Sep 23, 2020

Description

This feature will be done when the following UTM values are added:

  • tweeted 2x/day automatically
    utm_source=hypercharge-auto&utm_medium=twitter&utm_campaign=[bounty-title-name]

  • gitcoinbot will write a pinned post about it
    utm_source=hypercharge-auto-pinned-post&utm_medium=gitcoinbot&utm_campaign=[bounty-title-name]

  • there will be secret offer for the bounty
    utm_source=hypercharge-auto-secret&utm_medium=gitcoinbot&utm_campaign=[bounty-title-name]

  • there will be daily offer for the bounty -> No daily offers created in hypercharge mode
    utm_source=hypercharge-auto-daily&utm_medium=gitcoinbot&utm_campaign=[bounty-title-name]

  • The bounty will be featured in the daily email.
    utm_source=hypercharge-auto-daily&utm_medium=email&utm_campaign=[bounty-title-name]

  • Notify users who have worked with that org before
    utm_source=hypercharge-auto-notify&utm_medium=email&utm_campaign=[bounty-title-name]

  • Auto feature on hack explorer/normal prize explorer
    utm_source=hypercharge-auto-hack-explorer&utm_medium=gitcoin&utm_campaign=[bounty-title-name]

  • Auto feature on normal prize explorer
    utm_source=hypercharge-auto-prize-explorer&utm_medium=gitcoin&utm_campaign=[bounty-title-name]

image

Refers/Fixes

#7308

Testing

@zoek1
Copy link
Contributor Author

zoek1 commented Sep 23, 2020

As we talked @jdorfman yesterday, here is the UTM's for hypercharge mode :)

Copy link
Member

@thelostone-mc thelostone-mc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#7464 (review)

DRY code

@codecov
Copy link

codecov bot commented Sep 23, 2020

Codecov Report

Merging #7502 into master will increase coverage by 0.06%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #7502      +/-   ##
==========================================
+ Coverage   26.01%   26.08%   +0.06%     
==========================================
  Files         318      308      -10     
  Lines       31953    30780    -1173     
  Branches     4738     4562     -176     
==========================================
- Hits         8314     8029     -285     
+ Misses      23352    22477     -875     
+ Partials      287      274      -13     
Impacted Files Coverage Δ
app/dashboard/gas_views.py 22.77% <0.00%> (-0.70%) ⬇️
app/dashboard/models.py 49.57% <0.00%> (-0.14%) ⬇️
app/dashboard/notifications.py 17.29% <0.00%> (-0.09%) ⬇️
app/dashboard/router.py 40.34% <ø> (ø)
.../marketing/management/commands/hypercharge_mode.py 0.00% <0.00%> (ø)
app/economy/admin.py 64.28% <0.00%> (-5.42%) ⬇️
app/economy/utils.py 88.57% <0.00%> (-2.13%) ⬇️
app/marketing/mails.py 10.81% <0.00%> (-1.32%) ⬇️
app/grants/models.py 41.76% <0.00%> (-1.27%) ⬇️
app/taskapp/celery.py 93.33% <0.00%> (-0.79%) ⬇️
... and 35 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 819401a...d9e6d3f. Read the comment docs.

@thelostone-mc thelostone-mc marked this pull request as draft September 23, 2020 11:52
@jdorfman
Copy link

#7464 (review)

DRY code

@zoek1 are we able to reuse the code on https://gitcoin.co/hackathon-list?

@thelostone-mc
Copy link
Member

bump @zoek1 gimme a ping when you get to this and we can roll it out asap cause i know @jdorfman is waiting on ti

@zoek1 zoek1 marked this pull request as ready for review September 30, 2020 05:10
@zoek1
Copy link
Contributor Author

zoek1 commented Oct 1, 2020

this is also ready @thelostone-mc

@thelostone-mc thelostone-mc merged commit 9aafb39 into gitcoinco:master Oct 4, 2020
@jdorfman
Copy link

jdorfman commented Oct 5, 2020

via #7464

@jdorfman closing this out as this code is also present is open in PR https://github.com/gitcoinco/web/pull/7502/commits

@zoek1 @thelostone-mc I'm confused, the gas history page still has Disqus installed. ⬇️

image

@thelostone-mc
Copy link
Member

@jdorfman This hasn't been deployed yet !
Will roll out this week

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

Successfully merging this pull request may close these issues.

3 participants