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

그룹 엔티티(Group Entity) 를 수정하라 #46

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

yongjin5184
Copy link
Collaborator

@yongjin5184 yongjin5184 commented May 24, 2021

개요

  • 그룹에 필요한 엔티티를 수정하였습니다.

작업 내용

  • 모임 Entity 생성하라 에서 revert 로 인해 반영되지 못했던 부분을 반영하였습니다.
  • 모임 생성시, 설정하는 minimum_capacity (최소 인원) 과 maximumCapacity (최대 인원) 은 capacity (참여 인원) 필드로 수정하였습니다.

질문 및 논의 필요

스크린샷 2021-05-24 오후 1 48 49

-> identifier (long id -> Long id) 와 관련된 이야기 일 수 도 있는데, 같은 이유로 wrapper class 가 낫지 않나 제안드려봅니다. 😄

@truman-show
Copy link
Collaborator

마지막 커밋은 커밋 명이 잘못된거같습니다.

@truman-show
Copy link
Collaborator

해당 PR 내용은 현재 develop 브랜치에 머지가 된 내용일까요?

만약 그렇다면 close 처리 되어도 될거같아요

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants