Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci: 2023-08-31 정기배포 #719

Merged
merged 14 commits into from
Aug 31, 2023
Merged

ci: 2023-08-31 정기배포 #719

merged 14 commits into from
Aug 31, 2023

Conversation

jimin52
Copy link
Member

@jimin52 jimin52 commented Aug 31, 2023

개요

급작스럽게 정기배포합니다.
터지면 revert 예정

scarf005 and others added 14 commits August 22, 2023 17:14
* fix: `MYSQL_HOST`도 파싱

* feat: 백업, 연결, 테이블 생성 스크립트

* chore: `init/` 경로에서 로드

* chore: 로컬 db 초기화용 데이터

모든 개인정보가 제거된, book과 book_info만 존재하는 빈 덤프
* chore: add .gitignore in /docs

* docs: contract 작성법 추가
* feat: books contract 작성

* feat: 400, 500 에러 스키마 추가

* feat: 기부자 수정 api contract 추가

* refactor: 기부자 수정 api 404 에러 스키마 추가

* refactor: zod.extend 활용해 querySchema 중복 제거

* refactor: book status, z.enum 적용
* docs(ci): requirements.txt 사용

* docs: 이슈 및 브랜치 전략 링크로 정리
* build: ts-rest와 zod 버전 업데이트

* build: kysely 추가

* feat: db 타입 생성 및 kysely 연결

* feat: 타입 안전한 date 함수 래퍼

* refactor: `v2/reviews`에 kysely 적용

---------

Co-authored-by: nocontribute <>
* docs: 폴더로 정렬

* docs(chore): 사이트 꾸미기
* feat(books): 도서 추천 컨트롤러 추가

- 도서 추천 컨트롤러 추가
- books.routes.ts에서 도서 추천 라우터 순서 변경
- 도서 추천 컨트롤러 내에서 42 API 통신 테스트

* feat(books): 컨트롤러 내 함수를 서비스 함수로 분리

- access token을 가져오는 함수 서비스에 구현
- user의 42 API 상 id를 가져오는 함수 서비스에 구현

* feat(books): 사용자의 과제 정보 가져오는 모듈 추가

- 사용자의 과제 정보를 가져오는 서비스 함수 추가
- 사용자의 과제 정보를 담을 DTO 정의(추후 schema로 변경)

* feat(books): 사용자 login을 인자로 받아 프로젝트 받아오게 함

* feat(books): authvalidate 호출, req.user에 유저 정보를 담음

* feat: Project 타입에 marked_at 추가

* feat(books): 사용자의 project 정보 가져오는 컨트롤러 및 서비스 수정

- 사용자의 intraId를 DB에서 조회하게 변경
- access_token을 service의 전역변수로 설정
- access_token값이 만료될 경우, 재발급 후 42API와 통신
extendZodWithOpenApi와 zod 타입 사이 충돌 해결
* build: 전체 개발 모드 추가

* docs: 프로젝트 실행 방법 간략화

* docs: V1 명세 설명 추가
* feat(cursus): 과제 정보 가져오는 API 추가

- 과제 정보 가져오는 라우터 추가
- 과제 정보 관련 컨트롤러 추가

* feat: 42 API로부터 사용자의 과제 정보 받아오는 기능 구현 (#690)

* feat(books): 도서 추천 컨트롤러 추가

- 도서 추천 컨트롤러 추가
- books.routes.ts에서 도서 추천 라우터 순서 변경
- 도서 추천 컨트롤러 내에서 42 API 통신 테스트

* feat(books): 컨트롤러 내 함수를 서비스 함수로 분리

- access token을 가져오는 함수 서비스에 구현
- user의 42 API 상 id를 가져오는 함수 서비스에 구현

* feat(books): 사용자의 과제 정보 가져오는 모듈 추가

- 사용자의 과제 정보를 가져오는 서비스 함수 추가
- 사용자의 과제 정보를 담을 DTO 정의(추후 schema로 변경)

* feat(books): 사용자 login을 인자로 받아 프로젝트 받아오게 함

* feat(books): authvalidate 호출, req.user에 유저 정보를 담음

* feat: Project 타입에 marked_at 추가

* feat(books): 사용자의 project 정보 가져오는 컨트롤러 및 서비스 수정

- 사용자의 intraId를 DB에서 조회하게 변경
- access_token을 service의 전역변수로 설정
- access_token값이 만료될 경우, 재발급 후 42API와 통신

* refactor: getAccessToken() 함수 auth로 이동

* feat: 42API로부터 과제 정보 받아오는 API 변경

- index에 등록 및 라우터 반환값 수정

* feat(cursus): 42 API로부터 과제 정보 받아와서 저장하는 모듈 추가

- router 작성
- controller 추가
- service 추가

* feat(cursus): cursus의 서비스 파일 추가

* refactor: 변수명 path에서 uri로 변경

* feat(cursus): mode로 파일 덮어쓰기 및 붙여쓰기 결정

- mode 추가하여 파일 덮어쓸지 이후에 붙여쓸지를 결정
- json화 할 때 들여쓰기 적용

* feat(cursus): 빈 배열은 저장되지 않게 함

* feat(cursus): error 처리

* feat(cursus): error 시 반환 값 추가

* feat(cursus): 관련 DTO 추가

- ProjectFrom42 추가
- Campus 추가
- Cursus 추가

* feat: express-rate-limit 적용

- express-rate-limit 적용으로 분당 100회로 요청 횟수 제한

* feat: express-rate-limit을 books/recommand에만 적용

* feat(app): express-rate-limit 전체 적용

- express-rate-limit 전체 적용
- 오타 수정

* feat: rate limiter 호출 순서 변경

* feat: express rate limit 적용
feat: 인기 검색어 순위 집계 기능 및 swagger 구현
* refactor: 모듈명 컨벤션 지켜서 수정

* feat: 하루 한 번 최신 인기검색어 갱신을 위한 스케쥴러 도입

* refactor: 최신 인기 검색어 순위 집계 기준을 하루 늦추기

* refactor: 파일명 카멜케이스로 변경
add volume in compose database
@jimin52 jimin52 temporarily deployed to development August 31, 2023 02:43 — with GitHub Actions Inactive
@scarf005 scarf005 changed the title 08.31 정기배포 ci: 2023-08-31 정기배포 Aug 31, 2023
Copy link
Member

@scarf005 scarf005 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Member

@Jiwon-Woo Jiwon-Woo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jimin52 jimin52 merged commit c94196f into main Aug 31, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 완료
Development

Successfully merging this pull request may close these issues.

5 participants