Comments (16)
@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.
Related Issues (20)
- How this be used in an existing hyperledger composer angular2 project? HOT 1
- where can I find out serialized session in my hard disk? HOT 1
- req.logout doesn't delete the session HOT 1
- Got error when run npm install HOT 1
- README.md Sponser image is missing HOT 1
- user.password gives undefined HOT 1
- Database is insecure - passwords in plain text HOT 3
- Every connection log HOT 1
- it doesnt work at all HOT 2
- findById is relying on array position and not on user id HOT 1
- how to send custom response for auth failure
- duplicate query to the user table
- How to prevent user from going back using the browser? HOT 1
- says port is taken HOT 1
- http://localhost:3000/login/password Not found HOT 1
- Is passport.initialize() not needed? HOT 2
- Whats the meaning of cb?
- "ER_BAD_FIELD_ERROR: Unknown column 'NaN' in 'field list'" upon trying to sign up HOT 1
- Alice HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from todos-express-password.