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

기능 구현을 위한 예시 엔티티 설계 및 작성 #64

Merged
merged 7 commits into from
Dec 19, 2023

Conversation

stoneHee99
Copy link
Member

PR Type

돌아오는 주 부터 역할 분담 및 기능 작성을 위해 러닝커브가 있었던 부분 (Geometry 객체를 다루는 부분이라던지..)
을 감안하여 공부하고 예시 엔티티를 작성 및 리뷰 받고 싶어서 PR 날립니다!

  • 새로운 기능을 추가했어요.

Related Issues

#63

What does this PR do?

  • 단일 여행 지점의 정보를 저장하는 Spot 엔티티 설계 및 작성
  • 여러 여행 지점의 연결 코스를 저장하는 Course 엔티티 설계 및 작성
  • 사용자 화면에서 글로써 화면에 표시되는 단위인 Post 엔티티 작성 및 속성으로 필요한 상속 엔티티를 만들어 중복 방지

Other information

저도 학습을 하면서 한거라 엔티티를 이런 구조로 작성한 경험이 없어서 리뷰가 꼭 필요할 듯 합니다 (예상치 못한 문제가 있으면 안되니까요..ㅎㅎ)

@stoneHee99 stoneHee99 self-assigned this Dec 17, 2023
@stoneHee99 stoneHee99 linked an issue Dec 17, 2023 that may be closed by this pull request
5 tasks
Copy link
Collaborator

@minson96 minson96 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

Choose a reason for hiding this comment

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

asciidoctor.outputDir 은 환경별수에 별도로 등록하면 되는건가요?

Copy link
Member Author

Choose a reason for hiding this comment

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

환경변수에 등록하실 필요는 없을것 같아요 !!

Copy link
Collaborator

@ShineCorine ShineCorine left a comment

Choose a reason for hiding this comment

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

고생 많으셨습니다!. 새로운 어노테이션도 알게 됐고 rest docs 사용을 위한 설정도 파악이 되는 것 같습니다. 감사합니다.

@stoneHee99 stoneHee99 merged commit 1bac932 into develop Dec 19, 2023
1 check failed
@stoneHee99 stoneHee99 deleted the 63-기본엔티티설계 branch December 20, 2023 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

기본 엔티티 설계
3 participants