Skip to content
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

Node express microbyte #33

Closed
wants to merge 4 commits into from

Conversation

itherohit
Copy link

@itherohit itherohit commented Oct 23, 2020

Closes #32

Checklist:

  • Added activities for learners
  • Didn't give the solutions to the added activities straight away
  • Followed a LearnByDoing approach
  • Clean and well documented code (if source code files are involved)

@sudhanshutiwari264
Copy link

@itherohit Your pr is under review!

Copy link
Member

@YourAKShaw YourAKShaw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @itherohit!

About the current state of your PR:

  • It seems quite incomplete since it has only a README.md and no images or src files for reference. A big no no.
  • You README.md gives the vibe of a mere tutorial, which we are kinda against.

Actually the thing is that we at @CrioDo are sole believers of the fact that learning by doing is the best way to learn instead of following well laid out tutorials. What I see in you PR is that, you've got a README.md and in that you've laid out all of the required steps to accomplish a task. That's not the way to do it, at least not our way.

So, coming to the point, what needs to be done then?

A learn by doing content is one that virtually forces the learner to get out of her comfort zone and find solutions on her own. That is how the layout of your MicroByte should be. Instead of giving away the solutions, give hints, references, as to how the solution can be found. Kindly don't lead the learner to another tutorial that gives all the solutions straight away, that'll also be a big no. But, you're allowed to refer them to some links that'll give them some of the answers if not all.

I hope this makes you think more clearly about your approach. Expecting a great MicroByte from you.

Cheers. 🍻

@Kevinpaulose05
Copy link
Member

Kevinpaulose05 commented Nov 1, 2020

As we are done with #IBelieveinDoing - Hacktoberfest edition we need to close this PR as it is incomplete according to our guidelines. We are sorry to do this but we have to close this PR to avoid any further confusion on our side. We apologise for not being to guide you in the right time. Hope you participate actively in our next month's edition of #IBelieveinDoing and other events at Crio too. Till then keep contributing and all the best!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Node and Express
4 participants