codepathreview
commented
8 years ago When I run the code in your project to reproduce the error above, all Requireds are working. I'm going to mark this as complete. Are you still seeing the TypeError: Invalid attempt to destructure non-iterable instance error locally?
:+1: nice work. The goal of this assignment was to learn how to implement local authentication, interact with a data store, and render a webUI. In addition, you explored using middleware to faciliate common HTTP functionality. Over the next few weeks, we'll be focusing on more complex integrations and realtime functionality that we find in modern APIs and web apps.
Here's a checklist of things that I'm looking for in this project. You should review the checklist, and make any changes to your project, if necessary.
- Review the properties and methods that you've made available through your various
module.exports. Your modules should reveal the minimum interface and only expose things that must be used by other modules. - There should be no
*Synccalls anywhere in your code, and norequirecalls except at the top of each module file. - All required dependencies should be declared in the package.json
"dependencies". When installing dependencies, use the--saveoption:npm install --save somepackage. - Keep your code DRY by refactoring all code interacting with
reqorresinto a reusable composable middleware:(req, res, next)=>{}. - Your Models should exist each in their own module file, and Mongo embedded objects should have their own Schema. It is okay to include an embedded object's schema in the same file if it's not used in other schemas.
- Be sure to validate all user inputs. First, through strict enforcements in your data schemas, and second in your user, post and comment creation routes. Never rely solely on your schema for validation.
- All routes requiring a user account should enforce authentication through the
isLoggedInor equivalent route middleware:app.post('/post/:postId', isLoggedIn, ...) - When handling user authentication data, security is paramount. Never store plain-text passwords anywhere, and always be sure to use at least a global salt (pepper) when hashing.
Spend a lot of times to debug, but could not complete due to issues:
TypeError: Invalid attempt to destructure non-iterable instance at c:\Users\fnguye1\Desktop\Node\My-Projects\assignment3\routes.js:5:586 at Object.callee$1$0$ (c:/Users/fnguye1/Desktop/Node/My-Projects/assignment3/routes.js:136:64) at tryCatch (c:\Users\fnguye1\AppData\Roaming\npm\node_modules\babel\node_modules\regenerator\runtime.js:61:40) at GeneratorFunctionPrototype.invoke as _invoke at GeneratorFunctionPrototype.prototype.(anonymous function) as next at GeneratorFunctionPrototype.invoke (c:\Users\fnguye1\AppData\Roaming\npm\node_modules\babel\node_modules\regenerator\runtime.js:136:37) at run (c:\Users\fnguye1\AppData\Roaming\npm\node_modules\babel\node_modules\core-js\modules\es6.promise.js:104:47) at c:\Users\fnguye1\AppData\Roaming\npm\node_modules\babel\node_modules\core-js\modules\es6.promise.js:115:28 at flush (c:\Users\fnguye1\AppData\Roaming\npm\node_modules\babel\node_modules\core-js\modules\$.microtask.js:19:5) at nextTickCallbackWith0Args (node.js:433:9) at process._tickDomainCallback (node.js:403:13)