open-learning-exchange.github.io icon indicating copy to clipboard operation
open-learning-exchange.github.io copied to clipboard

Creating a diagram

Open bnmounir opened this issue 6 years ago • 12 comments

Problem (more of a suggestion)

If you think it's a good idea I would love to hear from you, what would be the most important commands that should be listed here and I will work on it. (also if you have a better idea about how it should be designed, for better understanding)

Screenshots

diag

Proposed solution

adding a diagram in GitHub steps listing the commands needed for most ops, to make it easy for junior devs to navigate the project.

bnmounir avatar Feb 20 '19 19:02 bnmounir

@bnmounir It will be good to have git workflow we follow instead of command flow.

singharpita avatar Feb 21 '19 01:02 singharpita

@bnmounir Yes as @singharpita said you. It will be nice to have block diagram of workflow. We want people to learn commands and also workflow. If we put commands on image they will just copy paste it.

Also please make a note that its our preferred workflow for this project. Because there are different workflow of git followed by different projects.

lmmrssa avatar Feb 21 '19 09:02 lmmrssa

hi @lmmrssa @singharpita, I see your point, I was just trying to come up with a picture showing the data flow, especially people without a lot of experience such as myself, I drew a little diagram to clear my confusion (what's on the computer VS what's on Github, also how to sync everything to avoid conflicts) and I thought it might be helpful for others.

bnmounir avatar Feb 22 '19 20:02 bnmounir

Sure @bnmounir that's why we wanted you to create block diagram of gitflow. Command is something they have to learn and understand not just copy paste. So flow diagram is something you can come up with

lmmrssa avatar Feb 23 '19 14:02 lmmrssa

hey @lmmrssa what do you think of this one, I think it draws a mental image of what's happening.

diag2

bnmounir avatar Feb 25 '19 17:02 bnmounir

@bnmounir I don't get the concept of State1 and State2. Because you are making fork from State1 creating branch then sending that request as State2, which looks like you wanted to say State is two versions of code. But later you are fetching from state2. Also pull request is from work branch to upstream, while looking at image it looks like it goes to master branch and then from there go to upstream.

If you are familiar with flowchart concept then making one that shows all steps will be nice.

lmmrssa avatar Feb 26 '19 05:02 lmmrssa

@lmmrssa I hear you, it would be really helpful if you could please list a few words you would like to see in this workflow, and I'll work on a sketch.

bnmounir avatar Feb 28 '19 16:02 bnmounir

@bnmounir words will be something similar. But just the way of presentation needs to be changed. Such as

  • Fork repository
  • Create new branch for issue from master branch
  • Commit changes
  • Create pull request
  • Sync fork repository But all these needs to be shown as flowchart and there could also include other stuff not mentioned here.

lmmrssa avatar Mar 02 '19 09:03 lmmrssa

I think the idea of a diagram is great! I would suggest making the theme color the blue that is already used in the tutorial notes. That way the page will look more seamless. For example:

Screen Shot 2019-03-25 at 2 57 46 PM

This might help: Screen Shot 2019-03-25 at 3 03 58 PM

zelmi avatar Mar 25 '19 19:03 zelmi

I think this might still be a good idea if someone wants to work on it. It depends on what exactly we're including in this diagram though, as there are some diagrams and explanations in the tutorial already.

SNutakki avatar Jun 08 '19 19:06 SNutakki

How about these drawings? Do they help?

  1. Fork and rename git1

  2. Create a branch git2

  3. Change content and Pull Request git3

  4. Pull request is approved and your branch is merged git4

hiroTochigi avatar Jun 16 '19 03:06 hiroTochigi

I like the idea! Diagrams are much easier to understand than words.

pattanawadee88 avatar Aug 29 '19 01:08 pattanawadee88