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

Merge: 북마크 생성, 조회, 삭제 적용 #44

Merged
merged 5 commits into from
Feb 26, 2024
Merged

Merge: 북마크 생성, 조회, 삭제 적용 #44

merged 5 commits into from
Feb 26, 2024

Conversation

Zamoca42
Copy link
Member

PR 체크리스트

아래 항목을 확인해 주세요:

  • 커밋 메시지가 우리의 가이드라인을 따르고 있는지 확인하세요
  • 변경 사항에 대한 테스트가 추가되었는지 확인하세요 (버그 수정 / 기능 추가)
  • 문서가 추가되거나 업데이트되었는지 확인하세요 (버그 수정 / 기능 추가)

PR 유형

이 PR은 어떤 종류의 변경을 가져오나요?

  • 버그 수정
  • 새로운 기능 추가
  • 코드 스타일 업데이트 (서식, 로컬 변수)
  • 리팩터링 (기능 변경 없음, API 변경 없음)
  • 빌드 관련 변경
  • CI 관련 변경
  • 문서 내용 변경
  • 애플리케이션 / 인프라 변경
  • 기타... 설명:

관련 이슈

이슈 번호: #42

현재 동작은 무엇인가요?

새로운 동작은 무엇인가요?

  • 문제id와 history를 배열로 받아 다수의 북마크 생성

  • 문제 테이블을 북마크 모델로 수정

    • favorite, wrong_question 테이블 삭제
    • history가 없으면 즐겨찾기, history가 있으면 틀린문제로 분류
  • 북마크 생성 시 문제가 이미 있다면 update, 없으면 insert

  • 북마크 생성, 조회, 삭제 시 유저 정보를 받아오는 pipe

  • 북마크 생성 dto


이 PR은 호환성 변경을 도입하나요?

  • 아니요

기타 정보

토익 문제 id로 북마크 생성

스크린샷 2024-02-26 오후 3 23 41

해당 유저에 이미 문제 id가 존재한다면 history만 업데이트

스크린샷 2024-02-26 오후 3 23 54

스크린샷 2024-02-26 오후 3 24 09

문제 삭제

스크린샷 2024-02-26 오후 3 24 24

스크린샷 2024-02-26 오후 3 24 44


- prisma를 이용하여 문제 테이블을 북마크 모델로 수정
- 북마크 모듈을 생성하고, 컨트롤러와 서비스를 작성

Related to #42
- favorite, wrong_question 테이블 삭제
- bookmark 테이블로 변경
- bookmark에서 history 컬럼 추가
- history가 없으면 즐겨찾기, history가 있으면 틀린문제로 분류

Related to #42
- 북마크 생성 시 문제가 이미 들어 있다면 update
- 없으면 insert 하도록 구현
- 북마크 조회 시 해당 유저의 북마크 리스트를 반환
- 북마크 삭제 시 해당 북마크를 삭제
- 북마크 생성, 조회, 삭제 시 유저 정보를 받아오는 pipe 구현
- 북마크 생성 dto 구현

Related to #42
- Swagger에서 nested reqeust body 프로퍼티가 보이게 변경
- 토익 문제 서비스 로직에서 메서드에 불필요한 async 제거
- workbook 파이프에서 upload 서비스를 사용하는 방식 변경

Related to #42
- 북마크 테스트에서 모듈 불러오기

Related to #42
@Zamoca42 Zamoca42 changed the title Feat/#42 Merge: 북마크 생성, 조회, 삭제 적용 Feb 26, 2024
Copy link
Contributor

@sheepdog13 sheepdog13 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

@Zamoca42 Zamoca42 merged commit 5fd1d82 into main Feb 26, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants