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

Origin/feat/profile #1

Merged
merged 9 commits into from
Nov 1, 2024
Merged

Origin/feat/profile #1

merged 9 commits into from
Nov 1, 2024

Conversation

Preta3418
Copy link
Owner

closed #이슈번호

#️⃣연관된 이슈

☑️ PR 유형

  • 새로운 기능 추가
  • 버그 수정
  • 문서 수정
  • 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
  • 코드 리팩토링
  • 테스트 추가, 테스트 리팩토링
  • 빌드 부분 혹은 패키지 매니저 수정
  • 주석 추가 및 수정
  • 파일 혹은 폴더 삭제
  • 파일 혹은 폴더명 수정
  • CSS 등 사용자 UI 디자인 변경

📝 작업 내용

✅ 피드백 반영사항

💬 리뷰 요구사항

deveunhwa and others added 9 commits October 30, 2024 08:14
본 Repository일 때만 워크플로가 실행하도록 수정합니다.
CD가 릴리스 브랜치에서만 이루어지도록 수정합니다.
기본적인 CRUD 기능을 탑재한 Profile 기능을 구현하였습니다. 현재 Member 마이그레이션이 종료되지 않아 실제 연결과 테스트 밑 Member 측에 추가 구현해야 하는 기능은 아직 작업하지 않았습니다

Related to: prgrms-be-devcourse#10
Service 단 인터페이스 생성 후 분리 시켰습니다

Related to: prgrms-be-devcourse#10
Member 측과 연결구조 를 추가했습니다. 연결 자체는 문제 없이 일어나며 현재 createProfile 을 DTO 를 통해서 받아오게 했는데 수정 예정 입니다

Related to: prgrms-be-devcourse#10
Profile 자동 생성을 구현하였습니다. 이제 Member 객체 생성시 Profile 또한 자동으로 생성 됩니다. 이 기능의 추가와 함께 일반적인 createProfile 코드를 지우지는 않았지만 순수하게 테스트 용도로만 쓰일 예정입니다.

Member 객체 생성시 자동으로 Profile 객체 또한 생성하도록 기능을 만들었습니다. 이에 따라 기존 createProfile 은 일반 유저에게 제공되는 기능이 아닌 테스트용 기능이 되었습니다

Related to: prgrms-be-devcourse#10
DTO 를 통해서 member 정보를 들고오는 기능이 의미가 없다 판단되어 repository 를 통해  member 정보를 들고오게 했습니다

Related to: prgrms-be-devcourse#10
@Preta3418 Preta3418 merged commit c279ae8 into main Nov 1, 2024
2 checks passed
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.

3 participants