[x] 로그인 없이 접근하면 안 되는 페이지에 Auth token 없이 접근했을 때 에러 객체와 함께 status code 200이 반환됨 401로 바꿔주세요 🔴 - PR 완료
[x] 모든 에러 형식이 통일되어 있나요?? 🔴 - PR 완료
uri 규격 통일
ex) 그룹 멤버 조회 등은 url 쿼리로 team id가 포함되는데 그룹 멤버 추가 등은 team id와 member id가 둘다 body에 있음. url 의미상 team id는 쿼리로 포함되는 게 맞는듯. 대신 url 조작은 누구나 가능하니 백엔드에서 방어로직 잘 짜야할듯!
[x] 민진
[x] 정구
[x] 지연
‘내 정보’를 이용하는 API들은 요청을 보낸 유저 식별을 Auth Token으로 하도록 변경 🟠
[x] 민진
[x] 정구
[x] 지연
각 도메인 간, ID 속성명 통일시키기 (프론트에서 타입스크립트 관련해서 처리가 쉬워짐) 🟠
ex) memberId vs userId, teamId vs groupId
ex) id vs memberId, id vs teamId,
// 예시 - 변경 전
{
"teamId": 38,
"name": "1"
}
// 변경 후
{
"team" : {
"teamId": 38,
"name": "1"
}
}
[x] 민진
[x] 정구
[x] 지연
데이터 추가/수정 후에는 추가한 데이터 반환
필수는 아닌 것 같고! 개인적으로 00 완료했습니다 메시지보다는 데이터를 반환받을 수 있었으면 프론트가 좀 더 편했을 거 같아요! 🟠
ex) 유저 초대 API /api/invite (팀 ID랑 멤버 ID를 받음)
프론트에서 inviteUser(teamId, members) 라는 함수에서 해당 API를 호출하고 초대한 멤버를 멤버 리스트에 추가하고 싶은 경우, 초대한 멤버를 리턴해주면: 인자 members에는 실제 API에 넘겨줄 멤버 ID만 넘겨주면 됨. 이후 리턴값을 그대로 리스트에 추가.
도메인 공통
[x] 로그인 없이 접근하면 안 되는 페이지에 Auth token 없이 접근했을 때 에러 객체와 함께 status code
200
이 반환됨401
로 바꿔주세요 🔴 - PR 완료[x] 모든 에러 형식이 통일되어 있나요?? 🔴 - PR 완료
uri 규격 통일
‘내 정보’를 이용하는 API들은 요청을 보낸 유저 식별을 Auth Token으로 하도록 변경 🟠
각 도메인 간, ID 속성명 통일시키기 (프론트에서 타입스크립트 관련해서 처리가 쉬워짐) 🟠
memberId
vsuserId
,teamId
vsgroupId
id
vsmemberId
,id
vsteamId
,데이터 추가/수정 후에는 추가한 데이터 반환