-
Notifications
You must be signed in to change notification settings - Fork 61
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
Add an alumni section to the contribute page #238
Comments
💡 idea: |
Maybe we do this issue first, and then create a separate issue for CONTENT in the editorial repo? Because I think this can be done by someone who doesn't necessarily like writing :) |
Agree :) |
Hi! I'm happy to take this one. |
@SonjaHeinen Great :) I've updated the post to show that you are working on this. Let us know if you have any questions! |
@finnp Thanks. Well, I'm pretty sure the code I just wrote is fine. However, I'm running into quite strange Github magic that makes the changes look like a brilliant mess. Not sure if I should submit anyway..? |
Hey @SonjaHeinen, this looked like a Wat to me too, but it's not a problem. It's just that GitHub detected the changes differently than you actually did. The end result is the same. If you delete lines 28 to 45, it is the same result as deleting 31 to 48. This is because the lines 28 to 30 are the same as 46 to 48. Do you understand what I mean? I'm typing this in mobile, so sorry if my explanation is a bit off. So yes you can submit it anyway :) |
@finnp Yeah. I figured that's what happened. In that case, I'm curious how the code review turns out ;-) |
@SonjaHeinen hey Sonja, did you submit a pull request? Do you need any help to finish things up? |
@gr2m Thought I made a pull request weeks ago o_0 |
@SonjaHeinen it’s here: https://github.com/SonjaHeinen/hood.ie/tree/patch-1 I think you missed to click the "Start Pull Request" button at the end, it’s a bit confusing, I miss that too at times |
I would like to work on this. |
okay all yours :) |
Part of #234
Starter Issue
We would like to take the opportunity to invite someone who did not yet contribute to Hoodie to work on this issue. Follow @YourFirstPR on Twitter for more of these issues across many open source projects☺️
💻 The Task
In preparation for the Camp Release we are trying to update the hood.ie site. This specific task involves updating the Hoodie community page to reflect our current contributors list. 🐶
COMMUNITY
andCONTRIBUTORS
Ping us in the Hoodie Chat or on Twitter if you have any questions :)
The text was updated successfully, but these errors were encountered: