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

♻️refactor: 패키지 구조 변경 #246

Merged
merged 24 commits into from
Mar 5, 2024
Merged

♻️refactor: 패키지 구조 변경 #246

merged 24 commits into from
Mar 5, 2024

Conversation

seheonnn
Copy link
Collaborator

@seheonnn seheonnn commented Feb 29, 2024 •
edited
Loading

☝️Issue Number

🔎 Key Changes

  • 1차적으로 도메인, 레포지토리 분리하였습니다
  • 이후 인프라 분리 진행하겠습니당

💌 To Reviewers

  • application-db.yml 의 local 부분이 ci 에서 에러가 나는 현상을 발견했습니다. 주석 처리하고 pr 하는 경우 잘 �통과되는 것을 확인하였고 로컬에서 실행 시 해당 부분 주석 해제 하시면 잘 작동합니다. 해당 에러는 조사 후 해결하겠습니다...

@seheonnn
💚 멀티모듈 세팅에 따른 ci 스크립트 변경
905aae6
@seheonnn
💚 ci 스크립트 로그 삭제
d6e3232
@seheonnn
💚 멀티모듈 세팅에 따른 ci 스크립트 변경
520aeac
@seheonnn
🐛 fix: root build.gradlew Jar 설정
12460bd
@seheonnn
🐛 fix: core 모듈 build.gradlew 파일 추가
823b686
@seheonnn
♻️refactor: 공고 이미지 업데이트 delete 메서드 추가
67286b8
@seheonnn
♻️refactor: S3 삭제 메서드 네이밍 변경
0a4eddb
@seheonnn
♻️refactor: db 모듈 분리
da937b7
@seheonnn
Revert "♻️refactor: S3 삭제 메서드 네이밍 변경"
4a96ecb 
This reverts commit 0a4eddb.
@seheonnn
🐛 fix: db 모듈 분리 원상복구
33a6966
@seheonnn
♻️ refactor: core 모듈 불필요한 의존성 삭제
55b6b8a
@seheonnn
♻️ refactor: DB 모듈 분리
81e2270
@seheonnn
🐛 fix: application-db.yml 디버깅
97b94d2
@seheonnn
🐛 fix: application-db.yml 디버깅
6a58e41
@seheonnn
🐛 fix: application-db.yml 파일 순서 변경
5be70fd
@seheonnn
🐛 fix: ci 파일 수정
9f5caca
@seheonnn
🐛 fix: db.yml 테스트
8c5aa1a
@seheonnn
♻️ refactor: ci.yml 파일 변경
bb74f72
@seheonnn
♻️ refactor: ci.yml 수정
7088992
@seheonnn
🐛 fix: db.yml local profile 주석 처리 및 ci.yml 수정
4db65ac
@seheonnn
♻️ refactor: 알림 허용 여부 컬럼 추가
4b213bd
@seheonnn
🐛 fix: db.yml local profile 주석처리
bd55cc4
@seheonnn
🐛 fix: db.yml local profile 주석처리
1b25a35
xxoznge
xxoznge approved these changes Mar 5, 2024
View reviewed changes
Copy link
Member

@xxoznge xxoznge left a comment

Choose a reason for hiding this comment

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

👍🏻

@seheonnn seheonnn merged commit 9aa1867 into develop Mar 5, 2024
1 check passed
@seheonnn seheonnn deleted the refactor/#245 branch March 5, 2024 06:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xxoznge xxoznge xxoznge approved these changes

@dbwp031 dbwp031 Awaiting requested review from dbwp031

@oxdjww oxdjww Awaiting requested review from oxdjww

@kimday0326 kimday0326 Awaiting requested review from kimday0326

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

️♻️ refactor: 패키지 구조 변경
2 participants
@seheonnn @xxoznge