✨ feat: add api for update emoji(group) and update api and update interface #34
Conversation
| req.body as GroupPostInputDTO, | ||
| ); | ||
| // 수정한 포스트 결과 | ||
| return res.status(201).json({ result: groupUpdatedPostResult, post: groupPostDetail}); |
There was a problem hiding this comment.
응답 시 result에 배열을 그대로 담아서 주는데, true or false값만 전달해주는게 어떤가요?
There was a problem hiding this comment.
groupUpdatedPostResult와 groupPostDetail 차이점이 뭔가요? 좀 헷갈려서..
There was a problem hiding this comment.
groupUp0datedPostResult는 수정 성공 여부이고 groupPostDetail은 수정 후 포스트의 정보를 전달하기 위한 해당 포스트의 디테일 정보입니다.
There was a problem hiding this comment.
result에 true false 값으로 수정하겠습니다.
| ); | ||
| // 비로그인 회원은 필요한 거- 비밀번호 확인 | ||
|
|
||
| if (groupUpdatedPostResult[0]===0) { |
There was a problem hiding this comment.
Sequelize UPDATE 시 return type이 [영향 받은 행 개수]니까 업데이트 수행 시 리턴값이 [0]이면 업데이트 할 포스트가 없다고 보고 이러한 조건식을 걸어두신건가요?
src/api/routes/v1/group.ts
Outdated
| export default (app: Router) => { | ||
| app.use("/groups", route); | ||
|
|
||
| // 개인 롤링페이퍼 포스트 생성 |
| req.body as GroupPostInputDTO, | ||
| ); | ||
| // 수정한 포스트 결과 | ||
| return res.status(201).json({ result: groupUpdatedPostResult, post: groupPostDetail}); |
There was a problem hiding this comment.
groupUpdatedPostResult와 groupPostDetail 차이점이 뭔가요? 좀 헷갈려서..
|
API 테스트 시 그리고, body에 담아 보내는 JSON도 camelCase로 수정해야할 것 같습니다! |
그룹 포스트 이모지 수정
- 이모지를 수정에 성공하면 JSON으로 수정 결과와 포스트 디테일 정보를 넘겨준다.
😀 이모지 사용
- 수정 실패 시 에러
위와 같이 포스트를 수정했을 때도 바꿔줌( 수정 후 포스트 디테일 정보 JSON으로 넘겨주기, 수정 실패시 에러)
인터페이스에서 snake -> camel로 수정
추가로 수정해야 할 부분: 개인 포스트 수정, 개인 포스트 이모지 수정 추가