Skip to content

Commit c813daf

Browse files
committed
added app.delete to /people:name route
1 parent 22e1b08 commit c813daf

File tree

1 file changed

+9
-2
lines changed

1 file changed

+9
-2
lines changed

classwork/server-2.js

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,8 +15,15 @@ app.put("/people", (request, response) => {
1515
response.json(people);
1616
});
1717

18-
app.delete("/people", (request, response) => {
19-
res.send("DELETE Request Called");
18+
app.delete("/people/:name", (request, response) => {
19+
const nameToDelete = request.params.name;
20+
const indexToDelete = people.findIndex((name) => name === nameToDelete);
21+
if (indexToDelete === -1) {
22+
response.status(404).send("Person not found");
23+
} else {
24+
people.splice(indexToDelete, 1);
25+
response.send("Person deleted");
26+
}
2027
});
2128

2229
const port = 3000;

0 commit comments

Comments
 (0)