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

Adds hackathon results to /results page #5069

Merged
merged 4 commits into from
Sep 4, 2019

Conversation

owocki
Copy link
Contributor

@owocki owocki commented Aug 25, 2019

Description

Adds hackathon results to /results page
#4944

Refers/Fixes
Testing

tested locallly:

Screen Shot 2019-08-25 at 4 30 58 PM

@@ -5118,3 +5118,11 @@
- Friends
- Love
image: 3box_butterfly.svg
- name: DAI
Copy link
Contributor Author

Choose a reason for hiding this comment

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

i dont know why this is on my PR. it already exists on master.

Copy link
Member

Choose a reason for hiding this comment

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

😓 Is github UI messed up again

Copy link
Contributor

Choose a reason for hiding this comment

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

Github has been letting us down lately... :(

Copy link
Contributor

Choose a reason for hiding this comment

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

Updating the PR w/ Master fixed it somehow...

@codecov
Copy link

codecov bot commented Aug 25, 2019

Codecov Report

Merging #5069 into master will decrease coverage by <.01%.
The diff coverage is 33.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5069      +/-   ##
==========================================
- Coverage   31.01%   31.01%   -0.01%     
==========================================
  Files         218      218              
  Lines       17531    17538       +7     
  Branches     2411     2413       +2     
==========================================
+ Hits         5437     5439       +2     
- Misses      11871    11874       +3     
- Partials      223      225       +2
Impacted Files Coverage Δ
app/retail/utils.py 10.12% <0%> (-0.06%) ⬇️
app/dashboard/admin.py 62.87% <100%> (ø) ⬆️
app/dashboard/models.py 56.55% <40%> (-0.05%) ⬇️
app/dashboard/views.py 14.08% <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 eee4030...269beee. Read the comment docs.

@owocki
Copy link
Contributor Author

owocki commented Aug 28, 2019

is this ok to merge? looks like it was approved but somehow didnt make the release...

@danlipert
Copy link
Contributor

@octavioamu does this look good to you?

Copy link
Contributor

@octavioamu octavioamu left a comment

Choose a reason for hiding this comment

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

The code looks good to me, I tried to run in my locally but even running create_page_cache and changing manually to check rinkeby bounties, wasn't able to get any hackathon on results page (looks like the command need some data I don't have).
Anyway I saw the information about stats on the hackathon event in admin.

@thelostone-mc thelostone-mc merged commit ff37104 into master Sep 4, 2019
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