Git Section Working with Remotes Lesson: The GitHub workflow SVG needs to be in the next lesson
Checks
- [x] This is not a duplicate of an existing issue (please have a look through our open issues list to make sure)
- [x] I have thoroughly read and understand The Odin Project Contributing Guide
- [x] Would you like to work on this issue?
Describe your suggestion
The "working with remote" lesson starts with the SVG of the workflow diagram of TOP repository(in order to contribute). But the appropriate place this picture would make sense will be the next lesson - "Using Git in Real Life" where the lesson actually talks about How to contribute to the curriculum. The picture has upstream repository in it which is introduced in the next lesson.
working with remote : https://www.theodinproject.com/lessons/javascript-working-with-remotes Using Git in Real Life : https://www.theodinproject.com/lessons/javascript-using-git-in-the-real-world
Path
Node / JS
Lesson Url
https://www.theodinproject.com/lessons/javascript-working-with-remotes
(Optional) Discord Name
No response
(Optional) Additional Comments
I was reading this lesson yesterday and I realized this problem. Since, I am currently on the lesson that tells me to contribute to the curriculum. I am very much excited to work on this issue(If I get it 😅). TOP has been so great and taught me a lot so far. I am grateful for that.
I agree, it would make more sense in the next lesson.
Thanks for making this issue and volunteering to contribute!
This issue is stale because it has had no activity for the last 30 days.
@StewardRighteous Do you still have any interest in putting in a PR for this issue?
Yess. I have interest in putting this PR
@StewardRighteous Do you still have any interest in putting in a PR for this issue?
Yess. I have interest in putting this PR. Sorry I had no idea how to reply to you, I thought my last reply would give you a notification.
This issue is stale because it has had no activity for the last 30 days.
Hi @StewardRighteous, thanks for putting this issue together. Given it hasn't had any movement in a couple of months, we'll let someone else have a crack at it.
Hi @KevinMulhern , I want to help with this PR.
I read the two lessons mentioned and in the event I am assigned this task, it would be necessary to specify if TOP wants to move only the picture or the whole section "Workflow diagram".
scenario 1
- If the entire section "Workflow diagram" is moved, the place indicated in the next lesson would be after "Commit messages for collaboration" section and before "Assignment" section.
- the first question in the Knowledge check needs to move as well.
- and the using remotes to change history lesson overview would need more context.
scenario 2
- If only the picture is moved, the new place in the next lesson will be the same, after "Commit messages for collaboration" section and before "Assignment" section, with the same title "Workflow diagram".
- in the initial lesson the title of the section "Workflow diagram" would need to be changed for a more appropriate one with the content that remained.
I hope I can help with this
Thanks @manelly67 - really appreciate how much thought you’ve already put into this. I’ve gone ahead and assigned it to you.
Just to double-check I’m on the same page: when you say "the entire section," are you including the content about git push --force as well?
Hi @KevinMulhern , thank you for the assignment.
Yes, when I said the entire section I am referring to including the content about git push --force. Because under the title "Workflow diagram" there are the two introductory lines, the image and the content git push --force.
In this order of ideas, in scenario 2 ( that looks like the most reasonable ) the git push --force content remains where it is, but giving the "git push --force" subtitle a new style as title format.
However, I would like to know: the aside section containing the lesson content, is not located in the curriculum repository. So, The markdown format change is sufficient for it to be read and included in aside section lesson content?. Because in both lessons the content would be changing.
Let me know your opinion to start working on this. Thanks
Thanks for clearing that up @manelly67 - I agree, scenario 2 sounds like the most reasonable approach and the way to go.
So, the markdown format change is sufficient for it to be read and included in aside section lesson content?
Yep, exactly! The aside content is automatically generated from the markdown based on the titles, so your markdown changes will take care of it.
Thank you @KevinMulhern , I will start working on this.