[feat] comment api 구현 #79

j03y14 · 2020-11-04T08:52:15Z

comment api 구현

내용

comment 추가 api 구현

309dd15
dff65e8
5595398

comment 추가 query 추가
comment 추가 model 메소드 추가
comment 추가 controller 메소드 추가
comment 추가 router 경로 추가

comment 가져오기 api

5595398
72ae49f
8f5a91d
9a003f1

comment 가져오기 query 추가
comment 가져오기 model 메소드 추가
comment 가져오기 controller 메소드 추가
comment 가져오기 router 경로 추가

comment 수정 api

53053a9
de638bf
f79b30e
a34c7dd

comment 수정 query 추가
comment 수정 model 메소드 추가
comment 수정 controller 메소드 추가
comment 수정 router 경로 추가

Why?

5595398

comment router가 url 상 issue의 하위에 들어가는 것이 맞다고 생각해서 issue router의 하위로 이동했습니다.

중간에 /:issueId/comment의 path variable이 req.params에 나오지 않는 문제가 있었는데 하위 라우터를 생성할 때

{ mergeParams: true }

옵션을 넣어주면 상위 라우터의 params들을 merge 한다는 것을 알게되었습니다.

comment를 추가하는 query를 추가했습니다.

comment model의 insert 메소드를 추가했습니다. paramArr에 무엇이 있는지 보여지도록 하는 것이 더 좋다고 생각해서 중간에 paramArr 변수로 만들어서 query에 넣었습니다.

comment router가 url 상 issue의 하위에 들어가는 것이 맞다고 생각해서 issue router의 하위로 이동했습니다. 중간에 /:issueId/comment의 path variable이 req.params에 나오지 않는 문제가 있었는데 하위 라우터를 생성할 때 mergeParams: true 옵션을 넣어주면 상위 라우터의 params들을 merge 한다는 것을 알게되었습니다.

comment select query를 추가했습니다. writer의 username도 가져오기 위해서 User table과 join을 해서 가져왔습니다.

comment model에 select 메소드를 추가했습니다. sql의 selectComment query를 이용했습니다.

comment controller read 메소드를 추가했습니다. 내부에서 commentModel.select() 메소드를 호출해서 값을 가져온 후 응답으로 보냈습니다. 중간에 오류가 있으면 500 에러를 보내도록 했습니다.

GET /api/issue/:issueId/comment로 들어오는 요청에 대해서 처리하도록 구현했습니다.

comment update query를 추가했습니다. comment_id가 같은 row의 content를 바꿔주도록 했습니다.

comment model update 메소드 추가를 했습니다. sql.updateComment를 사용했습니다.

comment controller의 update 메소드를 추가했습니다.

PATCH /api/issue/:issueId/comment/:commentId url에 대한 api를 추가했습니다.

통일성을 위해서 대문자를 소문자로 변경했습니다.

slogup-mia · 2020-11-04T09:16:55Z

web-BE/config/query.js

+
+  // comment
+  insertComment:
+    'INSERT INTO Comment(writer_id, content, write_time, is_issue_content, issue_id) VALUES (?,?,?,?,?)',


ㅋㅋㅋ user랑 comment랑 사이가 꽤 나쁜가보네용 🧐
9줄의 간격,,

conflict 때문에 그렇게 했었는데... 상관이 없나보네요

slogup-mia · 2020-11-04T09:23:32Z

web-BE/routes/api/issue/comment/index.js

@@ -0,0 +1,7 @@
+const router = require('express').Router({ mergeParams: true });


무한따봉.. 👍🏻👍🏻..

slogup-mia · 2020-11-04T09:26:38Z

web-BE/routes/api/issue/comment/controller.js

+      writer,
+      content,
+      write_teme: writeTime,
+      is_issue_content: isIssueContent,


~~is_issue_content: isIssueContent 에서~~
~~isIssueContent 가 0으로 할당되는 포인트를 못찾겠어요..!~~
~~코멘트면 백에서 처리해주어야 한다고 생각했는데, 어디에 있을까요 ?~~
줌 실시간 질문으로 이해했습니다!

SeunghyoKu

엄청난 Issue의 늪을 건너 comment까지 구현하신 재윤님..
칭찬 스티커 100만개 드리고 싶군요..
코드가 전체적으로 매우 깔끔해서 더 좋았습니다. 👍

SeunghyoKu · 2020-11-04T09:24:18Z

web-BE/routes/api/issue/comment/controller.js

+      res.status(200).json({ insertId });
+    } catch (err) {
+      console.error(err);
+      next(createError(500));


저번에도 비슷한 코멘트 남겼던 것 같은데 next(createError(~~~)) 간단하고 좋은 방법 같습니다 👀

SeunghyoKu · 2020-11-04T09:24:48Z

web-BE/routes/api/issue/comment/index.js

@@ -0,0 +1,7 @@
+const router = require('express').Router({ mergeParams: true });


모두를 충격에 빠트린 {mergeParams: true}...

SeunghyoKu · 2020-11-04T09:26:58Z

web-BE/config/query.js

@@ -27,4 +27,8 @@ module.exports = {
  // comment
  insertComment:
    'INSERT INTO Comment(writer_id, content, write_time, is_issue_content, issue_id) VALUES (?,?,?,?,?)',
+  selectComment:
+    'SELECT Comment.comment_id, Comment.writer_id, `User`.username, Comment.content, Comment.write_time, Comment.is_issue_content, Comment.issue_id '
+    + 'FROM Comment JOIN `User` ON Comment.writer_id = `User`.user_id WHERE Comment.issue_id = ?',


User는 User로 표현하셨는데, 이유를 여쭤 봐도 될까요?👀

SeunghyoKu · 2020-11-04T09:31:00Z

web-BE/routes/api/milestone/controller.js

@@ -18,7 +18,7 @@ const milestoneController = {
      const milestoneArr = await milestoneModel.select();
      res
        .status(200)
-        .json({ message: '마일스톤 읽기 성공', MilestoneArray: milestoneArr });
+        .json({ message: '마일스톤 읽기 성공', milestoneArray: milestoneArr });


ㅎㅎ.. 감사합니다..ㅎㅎ.. 😭

j03y14 added 11 commits November 4, 2020 17:22

[feat] insert comment query 추가

309dd15

comment를 추가하는 query를 추가했습니다.

[feat] comment model의 insert 메소드 추가

dff65e8

comment model의 insert 메소드를 추가했습니다. paramArr에 무엇이 있는지 보여지도록 하는 것이 더 좋다고 생각해서 중간에 paramArr 변수로 만들어서 query에 넣었습니다.

[feat] comment select query 추가

062e119

comment select query를 추가했습니다. writer의 username도 가져오기 위해서 User table과 join을 해서 가져왔습니다.

[feat] comment model select 메소드 추가

72ae49f

comment model에 select 메소드를 추가했습니다. sql의 selectComment query를 이용했습니다.

[feat] comment router get api 추가

9a003f1

GET /api/issue/:issueId/comment로 들어오는 요청에 대해서 처리하도록 구현했습니다.

[feat] comment update query 추가

53053a9

comment update query를 추가했습니다. comment_id가 같은 row의 content를 바꿔주도록 했습니다.

[feat] comment model update 메소드 추가

de638bf

comment model update 메소드 추가를 했습니다. sql.updateComment를 사용했습니다.

[feat] comment controller의 update 메소드 추가

f79b30e

comment controller의 update 메소드를 추가했습니다.

[feat] comment 수정 api 추가

a34c7dd

PATCH /api/issue/:issueId/comment/:commentId url에 대한 api를 추가했습니다.

SeunghyoKu self-requested a review November 4, 2020 09:00

slogup-mia self-requested a review November 4, 2020 09:14

j03y14 added the web-BE web backend label Nov 4, 2020

j03y14 self-assigned this Nov 4, 2020

[refactor] 대문자를 소문자로 변경했습니다

73ddbc9

통일성을 위해서 대문자를 소문자로 변경했습니다.

slogup-mia approved these changes Nov 4, 2020

View reviewed changes

SeunghyoKu approved these changes Nov 4, 2020

View reviewed changes

Merge branch 'web-BE' into web-BE-comment

44902e7

j03y14 merged commit 5fb1be5 into web-BE Nov 4, 2020

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[feat] comment api 구현 #79

[feat] comment api 구현 #79

j03y14 commented Nov 4, 2020

slogup-mia Nov 4, 2020

j03y14 Nov 4, 2020

slogup-mia Nov 4, 2020

slogup-mia Nov 4, 2020

SeunghyoKu left a comment

SeunghyoKu Nov 4, 2020

SeunghyoKu Nov 4, 2020

SeunghyoKu Nov 4, 2020

SeunghyoKu Nov 4, 2020

		@@ -0,0 +1,7 @@
		const router = require('express').Router({ mergeParams: true });

[feat] comment api 구현 #79

[feat] comment api 구현 #79

Conversation

j03y14 commented Nov 4, 2020

comment api 구현

관련 이슈 및 위키

내용

comment 추가 api 구현

comment 가져오기 api

comment 수정 api

Why?

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

SeunghyoKu left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment