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

Changes on hackathon onboarding #7149

Merged
merged 2 commits into from
Jul 27, 2020

Conversation

zoek1
Copy link
Contributor

@zoek1 zoek1 commented Jul 27, 2020

Description
  • Center sponsor logo when there is only one sponsor
  • Shrink up height of sponsor logo container by 30% when there is only one sponsor
  • Add ability to change border color on the onboarding page...group with this section in the admin
  • How does hackathon work section - add a collapse/expand arrow at the top of the section on right hand side

https://www.loom.com/share/60be4cbed58e44af9e943052969c62c2

Refers/Fixes

#7145

Testing

@codecov
Copy link

codecov bot commented Jul 27, 2020

Codecov Report

Merging #7149 into master will increase coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #7149      +/-   ##
==========================================
+ Coverage   26.29%   26.31%   +0.02%     
==========================================
  Files         300      300              
  Lines       29392    29393       +1     
  Branches     4322     4322              
==========================================
+ Hits         7728     7735       +7     
+ Misses      21393    21387       -6     
  Partials      271      271              
Impacted Files Coverage Δ
app/dashboard/models.py 48.63% <100.00%> (+0.01%) ⬆️
app/dashboard/embed.py 31.60% <0.00%> (+3.44%) ⬆️

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 f2f9b98...e8f4a26. Read the comment docs.

@danlipert danlipert merged commit d6b795a into gitcoinco:master Jul 27, 2020
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