Skip to content

Conversation

@GodUser1005
Copy link
Collaborator

@GodUser1005 GodUser1005 commented Jan 15, 2025

Sweepic Server PR List

close #37

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

  • 최신 커밋 업데이트

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

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

  • schema.prisma 파일 작성 및 RDS 마이그레이션 적용

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

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

  • schema.prisma 파일에 변경된 사항이 많습니다
    ERD 테이블에 맞게 기존의 ORM 모델들을 수정 및 생성 하였습니다.

    • User 모델 수정
    • SocialAccount 모델 생성(여럿 소셜로그인 구현을 위한 모델)
    • Notification 모델 생성(알림설정에 관한 내용 구현을 위한 모델)
    • Session 모델 생성(세션 로그인 기능을 위한 모델)
    • Image 모델 생성(이미지 정보에 대한 모델)
    • ImageTag 모델 생성(이미지-태그 의 다대다 관계를 위한 모델)
    • Tag 모델 생성(태그 기능 구현을 위한 모델)
    • TagCategory 모델 생성(장소태그, 인물태그 등 태그의 종류를 구분하기 위한 모델)
    • MemoFolder 모델 생성(메모장 폴더 기능을 위한 모델)
    • MemoImage 모델 생성(메모장에서 이미지로 저장 기능을 위한 모델)
    • Challenge 모델 생성(챌린지 기능 구현을 위한 모델)
    • ChallengeImage 모델 생성(챌린지-이미지 의 다대다 관계를 위한 모델)
    • LocationChallenge 모델 생성(위치기반 챌린지 기능을 위한 모델-Challenge 모델을 상속한 방식으로 구현)
    • DateChallenge 모델 생성(날짜기반 챌린지 기능을 위한 모델-Challenge 모델을 상속한 방식으로 구현)
    • Award 모델 생성(어워드 기능을 위한 모델)
    • AwardImage 모델 생성(어워드에 해당하는 이미지의 목록을 위한 모델)
    • Card 모델 생성(카드 기능을 위한 모델)
  • .src/models/user.model.ts 해당 파일에 정의된 user 타입 인터페이스를 변경한 스키마에 맞게 수정하였습니다.

  • ./prisma/migrations 내부의 여럿 파일들이 추가되고 수정 되었습니다.
    기존의 마이그레이션 내용과 테스트를 진행한 마이그레이션에 대한 내용이 들어있습니다.

🤚 동작 확인

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

  • 로컬에서 마이그레이션 적용
    image
    image

  • RDS에 마이그레이션 적용
    image
    image

  • yarn lint 적용시 결과
    image

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

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

  • 없음

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

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

  • RDS에 스키마에 해당하는 내용들이 적용되었습니다 혹시 어떤 테이블이 필요하다 혹은 이 테이블에 특정 칼럼이 있어야 할 것같다 라고 저에게 요청해주시면 스키마에 반영하고 마이그레이션 적용 시켜서 올려드리겠습니다.

🩺 이 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 the 📎 CHORE 초기 설정 및 패키지 설정 label Jan 15, 2025
@GodUser1005 GodUser1005 self-assigned this Jan 15, 2025
@GodUser1005 GodUser1005 linked an issue Jan 15, 2025 that may be closed by this pull request
Copy link
Collaborator

@codie0226 codie0226 left a comment

Choose a reason for hiding this comment

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

확인했습니다!

Copy link
Contributor

@jjiinaaa jjiinaaa 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
Collaborator

@Socializedistp Socializedistp left a comment

Choose a reason for hiding this comment

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

확인했습니다. 고생하셨습니다

Copy link
Collaborator

@asjasj3964 asjasj3964 left a comment

Choose a reason for hiding this comment

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

확인했습니다! 정말 수고 많으셨습니다!!

Copy link
Collaborator

@jonaeunnn jonaeunnn left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 !

@GodUser1005 GodUser1005 merged commit 161aad4 into develop Jan 16, 2025
1 check passed
@GodUser1005 GodUser1005 deleted the feature/SWEP-36 branch January 16, 2025 00:59
GodUser1005 added a commit to GodUser1005/sweepic-Server that referenced this pull request Jan 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📎 CHORE 초기 설정 및 패키지 설정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[SWEP-36] RDS 마이그레이션 적용 및 스키마 작성

7 participants