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

[바니] Chapter 4. DATABASE 설계 & AWS RDS 설정 #23

Merged
merged 2 commits into from
May 8, 2024
Merged

Conversation

suuu0719
Copy link

@suuu0719 suuu0719 commented May 6, 2024

#️⃣ 연관된 이슈

< #21 >

✅ 워크북 체크리스트

  • 모든 핵심 키워드 정리를 마쳤나요?
  • 핵심 키워드에 대해 완벽히 이해하셨나요?
  • 이론 학습 이후 직접 실습을 해보는 시간을 가졌나요?
  • 미션을 수행하셨나요?
  • 미션을 기록하셨나요?

✅ 컨벤션 체크리스트

  • 디렉토리 구조 컨벤션을 잘 지켰나요?
  • pr 제목을 컨벤션에 맞게 작성하였나요?
  • pr에 해당되는 이슈를 연결하였나요?
  • 적절한 라벨을 설정하였나요?
  • 스터디원들에게 code review를 요청하기 위해 reviewer를 등록하였나요?
  • 닉네임/main 브랜치의 최신 상태를 반영하고 있는지 확인했나요?

📌 주안점

@suuu0719 suuu0719 added the ✨ Chapter04 챕터 4 label May 6, 2024
@suuu0719 suuu0719 self-assigned this May 6, 2024
Copy link

@SoulTree-Lovers SoulTree-Lovers left a comment

Choose a reason for hiding this comment

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

이미지가 많아 글을 읽는데 도움이 많이 되었습니다.

Comment on lines +57 to +58
![image](https://github.com/SSUMC-6th/Spring_Boot_A/assets/118423039/e8171ef6-76b1-44d7-a815-2a809198bb78)
- ![image](https://github.com/SSUMC-6th/Spring_Boot_A/assets/118423039/ad216e98-42c3-454c-895a-c332ffb18ce9)

Choose a reason for hiding this comment

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

표기법 이미지를 통해 이해하기 수월했습니다!

Copy link

@bflykky bflykky left a comment

Choose a reason for hiding this comment

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

LGTM. ERD, 테이블 다이어그램을 볼 때 많이 쓰이는 새발 표기법에 대해 설명해준 게 좋았어요.

@h-ye-ryoung
Copy link

h-ye-ryoung commented May 7, 2024

키워드 정리도 이해하기 쉽게 해주셨고 ERD 설계도 깔끔하고 좋았습니다!!
다만 ERD에 대해 조금만 제 개인적 의견을 보태보자면..!

  1. store� 테이블에도 가게의 주소를 추가하면 좋을 것 같습니다!!
  2. 회원가입할 때 사용자가 취향을 선택하는 부분도 구현하면 더 좋을 것 같아요!!
  3. 생년월일은 보통 시간은 쓰지 않으니 datatime에서 date로 타입 바꾸면 좋을 것 같아요!
  4. 미션을 complete했을때 사용자에게 떨어지는(사용자에게 저장되는) 포인트도 저장하면 좋을 것 같습니다!

@suuu0719 suuu0719 merged commit 8f9a1b1 into bany/main May 8, 2024
@suuu0719 suuu0719 deleted the bany/#21 branch May 8, 2024 01:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants