Skip to content
This repository was archived by the owner on Aug 17, 2024. It is now read-only.

Conversation

@monahvi
Copy link

@monahvi monahvi commented Apr 11, 2023

Volunteers: Are you marking this coursework? You can find a guide on how to mark this coursework in HOW_TO_MARK.md in the root of this repository

Your Details

  • Your Name:
  • Your City:
  • Your Slack Name:

Homework Details

  • Module:
  • Week:

Notes

  • What did you find easy?

  • What did you find hard?

  • What do you still not understand?

  • Any other notes?


res.status(204).send("Message has been deleted!");
}
res.status(404).send("Message not found!");

Choose a reason for hiding this comment

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

Nice returning error messages and correct status codes 🥳

server.js Outdated
res.json({ messages });
});

app.get("/messages/id", (req, res) => {

Choose a reason for hiding this comment

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

Does this one work? I see in the others you have used a colon like :id

server.js Outdated
const cors = require("cors");

const app = express();
let app = express();

Choose a reason for hiding this comment

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

Why do we change this to let? I dont see it being reassigned

monahvi added 2 commits April 14, 2023 21:19
line 41 solved
change the let
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants