Use of process.nextTick() at db/users.js about todos-express-password HOT 16 OPEN

@Vincent440 Hey. @Xalior shared their opinion on the matter. Don't take it personally. I am sure it was not intentional. And please retract your comment. Let's keep the discussion clean. It's up to the maintainers to decide what to do.

from todos-express-password.

IMO, process.nextTick is not a widely known feature, so many may wonder why it's there at first glance, a more effective solution could just be putting the code inside a setTimeout, this is much more widely known and clearly let's the user know that the code is async.

from todos-express-password.

from todos-express-password.

There is a comment. Perhaps it is not clear to you :) but to me it’s saying it’s “faking an async action to verify it works in all cases”.

Hope that helps explain the use logic.

from todos-express-password.

@Xalior I was thinking maybe we could add some comment lines in code where it is used to make it clear to everyone including beginners like me.

from todos-express-password.

shrug is it the purpose of this repo to teach node generics?

How verbose a comment should be is open to endless debate, and fundamentally the choice of the comment writer :-)

from todos-express-password.

from todos-express-password.

@Xalior I understand that it's pretty much the basics of Node.js. The issue is that most developers(including me) are not aware of constructs like process.nextTick(). It would be helpful if we can add a one line comment to help clarify the subject.

from todos-express-password.

I can't agree more that some clarification would of saved me hours of dissecting that code months ago.

One comment to suggest what was happening there would of been a great improvement to what is supposed to be an example repository.

from todos-express-password.

from todos-express-password.

@Xalior It seems the behavior of process.nextTick() has changed since node v11. I was thinking of getting more knowledge on the usage of the subject from production level developers, that too of a library that is well renown world wide. It is true what you said that more information can be found via a simple google search. But the issue remains that most developers(including the authors of blogs on the web) are unaware or has poor understanding of the intricate working of node.js event loop. I just believed that by adding a good comment section in this part of the code will help enlighten everyone including me. I am not saying we should. It's just a suggestion. Cheers.

nodejs/node#22257
nodejs/node#22842
nodejs/nodejs.org#1804

from todos-express-password.

from todos-express-password.

@Xalior I agree with you that its not the job of a feature, function or framework specific tutorial to teach anyone generic node code. It's just a harmless suggestion to document the code well so that even beginners may grasp the concepts easily and hence lead to better usage of the library itself rather than messing around in the dark. Moreover, there is prevalent unawareness about the event loop within the node.js community. Cheers.

from todos-express-password.

Really? Because googling “what does node next tick do” gives a pretty reasonable answer on the first page... If you’re not able to google and look up one line of code you don’t understand maybe you should stop moaning at free tutorials and pay someone to teach you, because you certainly do not seem to be able to do it alone!
…
Sent from my iPhone
On 14 Apr 2020, at 03:43, Vincent Shury @.***> wrote:  I can't agree more that some clarification would of saved me hours of dissecting that code months ago. One comment to suggest what was happening there would of been a great improvement to what is supposed to be an example repository. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.

Get off open source if you are going to tell new coders they need to pay to learn more over requesting a single line comment for clarification on an irrelevant piece of code in a module.
you POS.

from todos-express-password.

from todos-express-password.

I apologize maybe there was a miscommunication sometime when you attacked me for adding my input, however I do not think your comment was at all appropriate as a response to my input on this issue.

I appreciate the work everyone has done to make Passport and these examples available, I am sorry I tried to add some input honestly.

from todos-express-password.