Feature/#3 #6

sss4920 · 2023-12-27T07:00:15Z

🚩 관련 이슈

close [Feat] 엔티티 설계 #3

📋 구현 기능 명세

엔티티를 간단하게 설계했습니다.

📌 PR Point

무슨 이유로 어떻게 코드를 변경했는지
관계성으로 맺긴했는데 필드중에 빠진게 있으면 추가하도록하겠습니당!
어떤 부분에 리뷰어가 집중해야 하는지
빠진거 있는지.

🛠️ 테스트

테스트

mmihye · 2023-12-27T13:39:25Z

linkmind/src/main/java/com/app/linkmind/domain/Toast.java

+@Getter
+@Entity
+@NoArgsConstructor(access = AccessLevel.PROTECTED)
+public class Toast {


링크 열람했는지 확인하는 변수 추가해야할것같습니다!

오 그렇네용 감사합니당 ☺️

mmihye · 2023-12-27T13:46:06Z

linkmind/src/main/java/com/app/linkmind/domain/Reminder.java

+@Entity
+@Getter
+@NoArgsConstructor(access = AccessLevel.PROTECTED)
+public class Reminder {


리마인더 엔티티에 리마인더 시점(날짜, 시간)을 따로 저장하지는 않나요?

사실 이부분에 대해서는 구현 할 때 어떻게 될지 몰라서 자료형도 좀 고민이 되었던 부분이네요 ㅜ 나중에 yyyy라든가 이런거 확정되고 나서 이부분은 변경해도 되지않을까..! 라는 생각이었습니당

나중에 변경해도 괜찮을거같아요!

mmihye · 2023-12-27T13:47:03Z

linkmind/src/main/java/com/app/linkmind/domain/Reminder.java

+	@JoinColumn(name = "category")
+	private Category category;
+
+	private String title;


title은 어떤것을 가리키는 변수일까용?

아 리마인더의 제목이 아니라 카테고리의 제목이었군용! 수정하겠습니당 ㅎㅎ

mmihye · 2023-12-27T13:54:29Z

linkmind/src/main/java/com/app/linkmind/domain/User.java

+@Entity
+@Getter
+@NoArgsConstructor(access = AccessLevel.PROTECTED)
+public class User {


이번주 열람한 링크랑 이번주 저장한 링크 개수도 저장해야할거같아요!

아 여기는 근데 소셜로그인 로직 끝나고 refactor 브랜치로 반영해보거나? 그 로직 구현할 때 필요할 때 추가하는 방식을 생각해봤는데 어떻게 생각하시나용?!

아아 넵 좋아요!!

… feature/#3

sss4920 added 2 commits December 27, 2023 15:43

feature: 도메인 설계

b2e5ceb

update: Toast 수정

a52b21c

sss4920 added the ⚒️ Feat label Dec 27, 2023

sss4920 requested a review from mmihye December 27, 2023 07:00

sss4920 self-assigned this Dec 27, 2023

Merge branch 'develop' into feature/#3

951b196

mmihye reviewed Dec 27, 2023

View reviewed changes

sss4920 added 3 commits December 28, 2023 16:19

refactor: 도메인 변경사항 반영

78d7438

Merge branch 'feature/#3' of https://github.com/Link-MIND/Server into…

462255b

… feature/#3

refactor: 토스터로 이름 변경

5f49b09

sss4920 merged commit d3977d1 into develop Dec 28, 2023
1 check passed

sss4920 deleted the feature/#3 branch December 28, 2023 13:06

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature/#3 #6

Feature/#3 #6

sss4920 commented Dec 27, 2023

mmihye Dec 27, 2023

sss4920 Dec 28, 2023

mmihye Dec 27, 2023

sss4920 Dec 28, 2023

mmihye Dec 28, 2023

mmihye Dec 27, 2023

sss4920 Dec 28, 2023

mmihye Dec 27, 2023

sss4920 Dec 28, 2023

mmihye Dec 28, 2023

Feature/#3 #6

Feature/#3 #6

Conversation

sss4920 commented Dec 27, 2023

🚩 관련 이슈

📋 구현 기능 명세

📌 PR Point

🛠️ 테스트

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment