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.

Sign up for GitHub

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

Jump to bottom

[CHORE] 도메인 패키지 구조로 변경 #6

Merged
merged 2 commits into from
Jul 7, 2024
Merged

[CHORE] 도메인 패키지 구조로 변경 #6

merged 2 commits into from
Jul 7, 2024

Conversation

eeddiinn
Copy link
Contributor

@eeddiinn eeddiinn commented Jul 7, 2024 •
edited
Loading

📌 관련 이슈

closed #5

📝 작업 내용

  • 패키지 구조로 되어있어서 초기에 회의한대로 도메인 구조로 변경하였습니다 !
  • 폴더를 domain( 비즈니스 로직 및 도메인 관련) , infra(인프라스트럭처 및 전역적으로 공통으로 사용되는 것 관련)로 나누었습니다.
    image

💬리뷰 요구사항

  • 혹시 잘못 삭제한 파일있는지 확인 해주세요 !!!
  • domain폴더 안에서 일단 제가 생각한대로 user, affiliation, university, notice, firstcome 으로 나누었는데 이 부분은 수정이 필요하다면 수정하셔도 됩니당

@eeddiinn eeddiinn added 🐰예진 🔥Pull Request This will not be worked on 👾CHORE 동작에 영향 없는 코드 or 변경 없는 변경 사항(주석 추가 등) labels Jul 7, 2024
@eeddiinn eeddiinn requested a review from softmoca July 7, 2024 08:30
@eeddiinn eeddiinn self-assigned this Jul 7, 2024
@softmoca
Copy link
Member

softmoca commented Jul 7, 2024

오오 너무 깔끔하네요 !! 좋습니다 ! 잘못된 파일이 안보여서 승인 하겠습니다 ~ !!
마지하구 저두 pull받아서 마저 작업 시작하겠습니답 !

@eeddiinn eeddiinn merged commit e71cea1 into main Jul 7, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@softmoca softmoca softmoca approved these changes

Assignees

@eeddiinn eeddiinn

Labels
🐰예진 👾CHORE 동작에 영향 없는 코드 or 변경 없는 변경 사항(주석 추가 등) 🔥Pull Request This will not be worked on
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[CHORE] 도메인 구조로 변경
2 participants
@eeddiinn @softmoca