Merge pull request #214 from learnsoftwaredevelopment/resolved-bug-in-post-software-api

JonathanLeeWH · web-flow · commit 1e47f484c313 · 2021-08-01T17:00:09.000+08:00
Resolved a bug in `POST` `software` API endpoint where the `software` model `meta` properties are not updated
diff --git a/controllers/api/softwareController.js b/controllers/api/softwareController.js
@@ -24,6 +24,7 @@ const postSoftware = async (req, res) => {
   const softwareObject = {
     ...body,
     meta: {
+      ...body.meta,
       addedByUser: userId,
       updatedByUser: userId,
     },
diff --git a/requests/api/softwares/patch_softwareById.rest b/requests/api/softwares/patch_softwareById.rest
@@ -1,4 +1,4 @@
-PUT http://localhost:3001/api/software/5f997876dde20a5a60ce1c64
+PATCH http://localhost:3001/api/software/5f997876dde20a5a60ce1c64
 Content-Type: application/json
 Authorization: bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJ1c2VybmFtZSI6InNhbXBsZSIsImlkIjoiNWY4ZGEzODcwODlmZmY0ZGIwNmJjMGFmIiwiaWF0IjoxNjAzODkzMzExfQ.ARy4EKDMzJqd4DcZyWXP5hygSQ9A2Bw1YfMQ-Nb03JU
 

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-PUT http://localhost:3001/api/software/5f997876dde20a5a60ce1c64`
	`1`	`+PATCH http://localhost:3001/api/software/5f997876dde20a5a60ce1c64`
`2`	`2`	`Content-Type: application/json`
`3`	`3`	`Authorization: bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJ1c2VybmFtZSI6InNhbXBsZSIsImlkIjoiNWY4ZGEzODcwODlmZmY0ZGIwNmJjMGFmIiwiaWF0IjoxNjAzODkzMzExfQ.ARy4EKDMzJqd4DcZyWXP5hygSQ9A2Bw1YfMQ-Nb03JU`
`4`	`4`