codestates / codestates/YourTurn

Not To Do - 작업하면서 틈틈히 체크해주세요

Open
#19 0 comments 0 reactions 2 assignees Claimed by @leo-labbb View on GitHub
backend E: 1h
Dominant language
JavaScript
Stars
0
Forks
0
PR merge metrics
No merged PRs in 30d

Description

### ISSUE
- Group: `server`
- Type:
- Detail: 사전 체크 리스트

### TODO
#### API
- [ ] 리소스 CRUD시 목적에 맞는 HTTP 메소드를 사용하지 않음
- [ ] 리소스 CRUD시 엔드포인트에 동사를 사용함
- [ ] 리소스 요청/응답시 필요한 json의 키/값 타입을 명시하지 않음
- [ ] 리소스 여러 개 조회 시 배열을 사용하지 않음
- [ ] HTTP 응답 코드의 사용이 적절하지 않음
- [ ] 응답 에러 메시지의 일관성이 없음
- [ ] 리소스 조회 시 json과 스키마 field간의 일관성이 없음
- [ ] 리소스 생성 시 신규 아이디를 리턴하지 않음
- [ ] 사용자 정보를 필요로 하는 요청에 토큰 또는 세션을 사용하지 않음
- [ ] 특정 사용자와 연관된 리소스 조회시 권한 문제를 고려하지 않음
- [ ] JWT 사용시 헤더에 토큰을 전달하는 내용을 명시하지 않음
- [ ] 엔드포인트가 spinal-case (소문자와 대시 문자열) 를 따르지 않음
- [ ] GET 요청에서 쿼리 파라미터 대신 HTTP body를 사용함
- [ ] HTTP body를 사용해야 하는 상황에서 쿼리 파라미터를 사용함

#### 스키마
- [ ] 스키마의 1:N, N:M 관계가 적절하지 않음
- [ ] 조인 테이블의 이름이, 관계있는 테이블 이름을 포함하지 않음
- [ ] 리소스의 CRUD에 필요한 테이블이 존재하지 않음

### Estimated time
>Pick one

`1h`

### Labels
- Estimated time: `E: 1h`
- Group : `server`
- Sprint: Sprint__4__
- Urgency: `High`

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.