Open saminasikander123 opened 9 years ago
Fixed errors. Ready for Review. /cc @codepath @codepathreview @CrabDude Thanks Adam for your help in finally figuring out the issues.
:+1: nice work. With this assignment, we've now explored all the main components for building REST APIs! If this were 2009, you would be well on your way to building most developer APIs that you could find published. Over the next few weeks, we'll be focusing on realtime APIs, tooling and production deployments, the last pieces in your arsenal for distributing your own enterprise APIs.
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.
module.exports
. Your modules should reveal the minimum interface and only expose things that must be used by other modules.*Sync
calls anywhere in your code, and no require
calls except at the top of each module file."dependencies"
. When installing dependencies, use the --save
option: npm install --save somepackage
.req
or res
into a reusable composable middleware: (req, res, next)=>{}
.isLoggedIn
or equivalent route middleware: app.post('/post/:postId', isLoggedIn, ...)
Had most of the features working. Then created a new twitter account and tried to use it. Since then the auth is broken and am not able to figure out why. Will continue to work on it. Attaching whatever I have so far. Video walkthrough is also attached to show the error. Thanks. @codepath @codepathreview