Skip to content

Conversation

@GodUser1005
Copy link
Collaborator

Sweepic Server PR List

close #214

⚒️develop의 최신 커밋을 pull 받았나요?

  • 최신 커밋 업데이트

🔍️ 이 PR을 통해 해결하려는 문제가 무엇인가요?

어떤 기능을 구현한건지, 이슈 대응이라면 어떤 이슈인지 PR이 열리게 된 계기와 목적을 Reviewer 들이 쉽게 이해할 수 있도록 적어 주세요
일감 백로그 링크나 다이어그램, 피그마를 첨부해도 좋아요

  • User 관련 기능들의 Tsoa 변환
  • Tag 관련 기능들의 Tsoa 변환
  • Image 관련 기능들의 Tsoa 변환
  • Trash 관련 기능들의 Tsoa 변환
  • 휴지통 기능 재조정

✨ 이 PR에서 핵심적으로 변경된 사항은 무엇일까요? (핵심 작업 내용)

문제를 해결하면서 주요하게 변경된 사항들을 적어 주세요

  • 위의 Tsoa 변환은 단순히 변환 작업이고 양이 너무 많기에 코드 변경사항은 생략하겠습니다.
    해당 변경사항은 코드를 참조해주세요

  • 휴지통 API 변경사항

    • 사진 복구 및 완전 삭제시 imageId가 아닌 mediaId를 받도록 수정하였습니다.

    • 위 로직으로 수정하였기에 휴지통 사진 조회 기능역시 삭제하였습니다.

    • 휴지통에 사진넣기는 기존과 동일하게 imageId로 요청을 받습니다.

    • 휴지통 관련 파일의 이름을 trust에서 trash로 수정하였습니다.

    • trash.controller.ts
      image
      image
      image

    • trash.service.ts
      image

    • trash.repository.ts
      image
      이미지의 상태를 변경하는 DAO로 복구와 휴지통에 넣기에 둘다 사용되는 함수입니다.

      image 이미지를 완전 삭제하는 DAO 입니다.
    • app.ts
      image
      기존의 라우터들은 전부 삭제하였습니다.

🤚 동작 확인

기능을 실행했을 때 정상 동작하는지 여부를 확인하고 스크린 샷을 올려주세요

  • yarn lint
    image

  • yarn compile
    image

  • /docs/v2
    image
    image
    image
    image
    image
    /docs/v2 의 경로를 요청하였을 때 모든 API가 Tsoa Swagger로 작성되었음을 확인할 수 있습니다.

🔖 핵심 변경 사항 외에 추가적으로 변경된 부분이 있나요?

없으면 "없음" 이라고 기재해 주세요

  • 없습니다

🙏 Reviewer 분들이 이런 부분을 신경써서 봐 주시면 좋겠어요

개발 과정에서 다른 분들의 의견은 어떠한지 궁금했거나 크로스 체크가 필요하다고 느껴진 코드가 있다면 남겨주세요

🩺 이 PR에서 테스트 혹은 검증이 필요한 부분이 있을까요?

테스트가 필요한 항목이나 테스트 코드가 추가되었다면 함께 적어주세요

📌 PR 진행 시 이러한 점들을 참고해 주세요

  • Reviewer 분들은 코드 리뷰 시 좋은 코드의 방향을 제시하되, 코드 수정을 강제하지 말아 주세요.
  • Reviewer 분들은 좋은 코드를 발견한 경우, 칭찬과 격려를 아끼지 말아 주세요.
  • Review는 특수한 케이스가 아니면 Reviewer로 지정된 시점 기준으로 2일 이내에 진행해 주세요.
  • Comment 작성 시 Prefix로 P1, P2, P3 를 적어 주시면 Assignee가 보다 명확하게 Comment에 대해 대응할 수 있어요
    • P1 : 꼭 반영해 주세요 (Request Changes) - 이슈가 발생하거나 취약점이 발견되는 케이스 등
    • P2 : 반영을 적극적으로 고려해 주시면 좋을 것 같아요 (Comment)
    • P3 : 이런 방법도 있을 것 같아요~ 등의 사소한 의견입니다 (Chore)


📝 Assignee를 위한 CheckList

  • To-Do Item

@GodUser1005 GodUser1005 added ♻️ REFACTOR 기능 향상 및 리팩토링 🐛 FIX 버그 수정 ✨ FEAT 기능 구현 labels Feb 15, 2025
@GodUser1005 GodUser1005 self-assigned this Feb 15, 2025
@jjiinaaa jjiinaaa merged commit 0d29092 into develop Feb 15, 2025
1 check passed
@jjiinaaa jjiinaaa deleted the refactor/SWEP-62 branch February 15, 2025 05:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 FIX 버그 수정 ♻️ REFACTOR 기능 향상 및 리팩토링 ✨ FEAT 기능 구현

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[SWEP-62] Tsoa 변환(User, Tag, Image, Trash), 휴지통 API 로직 수정

3 participants