-
Notifications
You must be signed in to change notification settings - Fork 18
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
feat(SponsorsTable): add gadfly #586
Conversation
✅ Deploy Preview for keen-banoffee-86ed50 ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
btw we have the sponsor section two time: one at the top and one just before the footer |
Ah, good catch @TheJoin95! I will remove it before merging this one :) |
So... a PR that should have been a one liner, ended up being a whole rewrite lol I think this was needed because there where some comments left over from past versions that were cluttering the boilerplate and the sponsors where added one by one instead of creating a loop to render them. Let me know what you think, the PR is mostly deletions. Also, any React feedback that you have on this will be greatly appreciated :) |
It's okay Pato, we need a new website for OSDay, let's just do the job for this edition then we'll see for the next one |
…ic variables for future readability
As title says