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

Update to Add styles and Hackathon project page #6569

Closed
wants to merge 34 commits into from

Conversation

zoek1
Copy link
Contributor

@zoek1 zoek1 commented May 4, 2020

Description

Original work Hackathon Project Page #6297

Minor error fixes and fix styles on the mobile view.

image

image

To move this fast I fixed some error on indexing and also fix some styles on mobile view, thanks to @walidmujahid for his awesome work on this feature.

Refers/Fixes
Testing

walidmujahid and others added 30 commits April 17, 2020 20:58
TODO: if no hackathon is ongoing, default to tribes who participated in
the most number of hackathons.
TODO: if no hackathon going, show tribes that most recently
listed their bounties.
add rows to help space elements out properly and fix grid sizs.
if current hackathons exists, the tab is defaulted to current.
if there are no current hackathons, the default tab is upcoming.
otherwise, the default tab is finish.
`get_hackathon_event`, `get_tribe`, `tribe_fields`,
and `hackathons_funded`
@walidmujahid
Copy link
Contributor

walidmujahid commented May 4, 2020

I have cherrypicked 6ce4794 into my PR.

As for commit 75281b6 I am not quite sure what is happening. If the SVG is being deleted, which I understand since it is unused, the linked asset should not changed from PNG in the template.

@walidmujahid
Copy link
Contributor

walidmujahid commented May 4, 2020

used 75281b6 and df3baf3 in b24217b

@thelostone-mc
Copy link
Member

@zoek1 + @walidmujahid can this be closed out ?
since 6297 has all the commits ?

@walidmujahid
Copy link
Contributor

walidmujahid commented May 4, 2020

@thelostone-mc Yes, #6297 has all the commits. I think 6569 can be closed? @zoek1?

@PixelantDesign
Copy link
Contributor

Thanks @zoek1!

@androolloyd
Copy link
Contributor

@zoek1 can we sort out travis?

@androolloyd androolloyd closed this May 4, 2020
@androolloyd androolloyd reopened this May 4, 2020
@walidmujahid
Copy link
Contributor

@zoek1 can we sort out travis?

Everything is sorted on the original PR @androolloyd #6297

@zoek1
Copy link
Contributor Author

zoek1 commented May 4, 2020

Sure, give a moment

@codecov
Copy link

codecov bot commented May 4, 2020

Codecov Report

Merging #6569 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #6569   +/-   ##
=======================================
  Coverage   27.21%   27.21%           
=======================================
  Files         289      289           
  Lines       26798    26798           
  Branches     3962     3962           
=======================================
  Hits         7293     7293           
  Misses      19238    19238           
  Partials      267      267           

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 8b20ffe...8b20ffe. Read the comment docs.

@zoek1
Copy link
Contributor Author

zoek1 commented May 4, 2020

Perfect @walidmujahid integrated my changes 🙌 Lets with #6297!

@zoek1 zoek1 closed this May 4, 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.

5 participants