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/payment #10

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

Feature/payment #10

wants to merge 5 commits into from

Conversation

hanbroz
Copy link

@hanbroz hanbroz commented May 22, 2019

git 개념상실

@greenn-lab
Copy link

greenn-lab commented May 23, 2019

게시판도 추가로 구상하시고, 고생 많으셨어요.

Code 관련 엔티티는 아직 안올라 온것 같네요.
회원 엔티티에서도 쓰는 부분이 있어, 연계가 필요할 것 같네요.

이전에 다른 분 코드에서 남긴 내용인데, 패키지 구조를 기본 구조 com.shopping.jpa 뒤에 도메인 대표 키워드를 붙여서 com.shopping.jpa.payment 이렇게 정하는게 좋을 것 같아요.

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.

2 participants