Skip to content

Conversation

@octavioamu
Copy link
Contributor

@octavioamu octavioamu commented Oct 6, 2019

Description

New onboard hackathon flow
send to mailchimp
Need list creation on mailchimp account and paste list id on MAILCHIMP_LIST_ID_HACKERS production env
image

https://secure.vidyard.com/organizations/14217/players/GUEbzb7o2qUo5cuv1CCwf5?edit=true

Refers/Fixes

Ref #5237

Testing

@codecov
Copy link

codecov bot commented Oct 7, 2019

Codecov Report

Merging #5303 into master will increase coverage by 0.01%.
The diff coverage is 36.53%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5303      +/-   ##
==========================================
+ Coverage   29.83%   29.85%   +0.01%     
==========================================
  Files         233      233              
  Lines       19453    19505      +52     
  Branches     2782     2785       +3     
==========================================
+ Hits         5804     5823      +19     
- Misses      13410    13440      +30     
- Partials      239      242       +3
Impacted Files Coverage Δ
app/app/urls.py 89.58% <ø> (ø) ⬆️
app/app/settings.py 79.19% <100%> (+0.07%) ⬆️
app/dashboard/admin.py 63.71% <100%> (+0.62%) ⬆️
app/dashboard/views.py 13.35% <13.51%> (ø) ⬆️
app/dashboard/models.py 50.25% <90%> (+0.18%) ⬆️
...rketing/management/commands/no_applicants_email.py 0% <0%> (ø) ⬆️

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 a8b7bd2...49d1430. Read the comment docs.

@danlipert danlipert marked this pull request as ready for review October 16, 2019 08:40
@thelostone-mc thelostone-mc merged commit a04dfea into master Oct 16, 2019
@thelostone-mc thelostone-mc deleted the capture-hackathon-onboard branch October 24, 2019 11:38
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.

4 participants