Skip to content

here we are#23

Open
asakleiman wants to merge 2 commits into
pce-uw-jscript400:masterfrom
asakleiman:master
Open

here we are#23
asakleiman wants to merge 2 commits into
pce-uw-jscript400:masterfrom
asakleiman:master

Conversation

@asakleiman
Copy link
Copy Markdown

No description provided.

Comment thread readme.md

* **Your Answer:**

a reference to the Express module, where all of our good server-running-stuff lives
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

our instance of the server, also a function / event emitter

Comment thread readme.md

* **Your Answer:**
It's a sign that has fallen over, here I can fix it: `|`
It's the path of the url being passed to the server.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🥇

Comment thread readme.md

* **Your Answer:**

Giant error. body-parser gives access to req.body
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍

Comment thread readme.md
* **Question:** Try creating a new vegetable. Then, try restarting your server. What happens to the data you posted and why?

* **Your Answer:**
the vegitable disappears, because it's not getting written to a file anywhere or otherwise persistently stored
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

👍

Comment thread readme.md

* **Your Answer:**

standarjs doesn't appear in my copy of the json file, but according to the link it's a linter and formatter (corrects sloppily formatted coding)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Enforces AirBnB style guides. Nice Work!

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.

2 participants