codestates / codestates/DEVzine-port
[DEV LOG] 2021.08.20 강하은
- Dominant language
- JavaScript
- Stars
- 1
- Forks
- 0
- PR merge metrics
- No merged PRs in 30d
Description
## Contributions
- DB update & reset (배포, 개발)
- models:
- contributions, articles 조회수 기본값 변경
- 모든 모델 --v 필드 미포함으로 변경
- users 모델 스택 기본값 변경
- seed:
- user 50명 추가, 총 64명
- articles 가장 최근 크롤링 된 데이터 업데이트, 기사 총 120개
- contributions Date 데이터타입 불일치 문제 해결
- 기사 조회수 바로 +1 안되는 문제 해결 (쿼리 문제)
- visual 전체 누적 키워드 집계
- FE 와 논의 : 기사 중 보안 키워드가 너무 많다는 건에 대해
- 원인: 현재 '보안뉴스'에서 받아오는 모든 기사가 '보안' 카테고리에 포함, 다른 매체에서도 '보안' 카테고리로 글 받아오고 있음
- 해결방안 1) 보안뉴스 기사 개수 줄이기? 현재 최대 10개임 → 6, 5개
- 해결방안 2) 보안뉴스 키워드도 세분화
- 결론: 1로 진행 후 시간 나면 2 진행
- 관리자 페이지
- 승인완료, 승인대기 기고글 조회
- 거부, 승인 api 에 400 에러코드 추가 (유효하지 않은 status code)
- 거부 200 메세지 수정 (update rejected → reject success)
- 거부 endpoint → 상태코드 확인 후 그에 맞게 '거부'로 변환 (120번대)
- 승인 endpoint → 상태코드 확인 후 그에 맞게 '승인'으로 변환 (110번대)
- 관리자 권한
- admin 용 passport strategy 만들어 req.user.admin 으로 구분
- 모든 라우터에 관리자용 토큰 인증 추가
- 개별 기고글 보기 페이지
- 관리자 권한 확인 위한 미들웨어 추가
- 관리자는 모든 기고글 볼 수 있도록 쿼리 수정
## Difficult point
- QA를 할수록 늘어나는 버그...
## TODO tomorrow
- QA 하며 발견한 버그 잡기
- DB 메모리 누수 관리
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.