Jasmeen/580 rewrite contribute instructions#1200
Conversation
|
paigem
left a comment
There was a problem hiding this comment.
Submitting this review as is since I need to end my work day. I'll come back and continue to review the last bit of material - section 3 onwards!
Overall, I really like this update! Thanks @KAUR1984!! I make several suggested edits here. Keen to hear your thoughts!
|
Thanks for adding your feedback @paigem! I don't have any alternative suggestions to add. For your last comment, I'm happy to include the actual steps, as the current version might be a little difficult for people who are new to Git. |
|
Currently refining these guidelines and will request another review from @paigem once the updates are complete. |
paigem
left a comment
There was a problem hiding this comment.
This is looking really good, thanks so much @KAUR1984! Most of these comments are pretty small/straightforward. I started reviewing the checklist before remembering that you said to ignore for now - sorry about that!
Let me know if you have any questions about any of my comments!
mvgarcia
left a comment
There was a problem hiding this comment.
When I was following the 'How to contribute' instructions in Section 2 (Edit a page directly), I found out that 'creating a new branch' and 'editing the text' happen in a different order for ACCESS-NRI members. I moved this instructions to the correct place.
Co-authored-by: Paige Martin <ms.paigem@gmail.com>
Co-authored-by: Paige Martin <paige.martin@anu.edu.au>
Co-authored-by: Paige Martin <paige.martin@anu.edu.au>
Co-authored-by: Paige Martin <paige.martin@anu.edu.au>
Co-authored-by: Paige Martin <paige.martin@anu.edu.au>
Co-authored-by: Paige Martin <paige.martin@anu.edu.au>
Co-authored-by: Paige Martin <paige.martin@anu.edu.au>
Co-authored-by: Paige Martin <ms.paigem@gmail.com>
Co-authored-by: Valentina García <36575846+mvgarcia@users.noreply.github.com> Co-authored-by: Paige Martin <ms.paigem@gmail.com>
Co-authored-by: Paige Martin <ms.paigem@gmail.com>
0f78a04 to
ba6a658
Compare
Fixes issue #580. First attempt for rewriting contribution instructions. Feel free to give as much feedback as possible. Currently, in progress with adding a few minor tweaks on the UI side.