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

[Feature] Entity 추가 #7

Merged
merged 3 commits into from
Jan 22, 2024
Merged

[Feature] Entity 추가 #7

merged 3 commits into from
Jan 22, 2024

Conversation

joowojr
Copy link
Contributor

@joowojr joowojr commented Jan 22, 2024

🔴 Summary

  • 전반적인 Entity 설계

🟡 Changes Made

  • Record (AIRecord, BasicRecord) entity 추가
  • HelpRecord entity 추가
  • User entity 전화번호 column 추가
  • User entity에 봉사 신청 목록 가져오는 코드 추가

🟢 Other Information

  • 모르겠는 것은 코멘트해주세요!

🔵 PR Checklist

Please check if your PR fulfills the following requirements:

  • The commit message follows our guidelines.
  • PR title is correctly formatted and labels are added.
  • The program operates correctly.

Copy link
Member

@kiwijomn kiwijomn 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 +35 to +45
@Column(nullable = false)
private Date date;

@Column(nullable = false)
private String placeId;

@Column(nullable = false)
private String district;

@Column(nullable = false)
private String address;
Copy link
Member

Choose a reason for hiding this comment

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

생각해봤는데 이 부분에 대한 값을 사용자가 보수 신청할 때 함께 요청하는 거면 Record로 옮겨야 할 것 같아요!
보수 신청 ui에서 주소랑 희망 일자를 같이 입력하는 구조면 이 값들까지 한번에 묶어서 Record 엔티티로 저장해야 하지 않을까 싶어서요 😶

Copy link
Contributor Author

Choose a reason for hiding this comment

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

생각해봤는데 이 부분에 대한 값을 사용자가 보수 신청할 때 함께 요청하는 거면 Record로 옮겨야 할 것 같아요! 보수 신청 ui에서 주소랑 희망 일자를 같이 입력하는 구조면 이 값들까지 한번에 묶어서 Record 엔티티로 저장해야 하지 않을까 싶어서요 😶

일반 저장(기록용)/ 수리 요청 이렇게 나눠서 저장하는걸로 얘기했던거 같아서 엔티티를 분리했어요!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

image
참고 화면입니당

Copy link
Member

Choose a reason for hiding this comment

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

확인했습니다!

@Entity
@Getter
@NoArgsConstructor(access = AccessLevel.PROTECTED)
@DiscriminatorColumn
Copy link
Member

Choose a reason for hiding this comment

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

유저 롤 타입도 이렇게 DType으로 일반 사용자랑 봉사자 구분해서 설정하는 걸로 생각하면 될까요?!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

유저 롤 타입도 이렇게 DType으로 일반 사용자랑 봉사자 구분해서 설정하는 걸로 생각하면 될까요?!

유저는 타입에 따라서 컬럼이 달라지지는 않는거 같아서 그냥 ENUM 타입으로 넣어도될거같습니다!

Copy link
Member

Choose a reason for hiding this comment

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

유저는 타입에 따라서 컬럼이 달라지지는 않는거 같아서 그냥 ENUM 타입으로 넣어도될거같습니다!

그럼 봉사자 신청할 때 개인 / 기관(단체) 나누지 않고 그냥 롤만 바뀌는 건가요??

Copy link
Contributor Author

Choose a reason for hiding this comment

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

유저는 타입에 따라서 컬럼이 달라지지는 않는거 같아서 그냥 ENUM 타입으로 넣어도될거같습니다!

그럼 봉사자 신청할 때 개인 / 기관(단체) 나누지 않고 그냥 롤만 바뀌는 건가요??

넵 저는 그렇게 생각했는데, �롤에 따라 추가될 컬럼이 있을까요?

@joowojr joowojr merged commit bb88df3 into develop Jan 22, 2024
1 check passed
@joowojr joowojr deleted the feature/entity branch January 22, 2024 16:02
@joowojr joowojr self-assigned this Jan 23, 2024
@joowojr joowojr added the ✨ feature 새 기능 추가 label Jan 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feature 새 기능 추가
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants