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

[BE] Member 엔티티 추가 및 MemberAction 삭제 #546

Open
3 tasks
Arachneee opened this issue Sep 9, 2024 · 0 comments
Open
3 tasks

[BE] Member 엔티티 추가 및 MemberAction 삭제 #546

Arachneee opened this issue Sep 9, 2024 · 0 comments
Assignees
Labels
⌨️ BE Backend 🚧 refactor refactoring
Milestone

Comments

@Arachneee
Copy link
Contributor

Arachneee commented Sep 9, 2024

📄 설명

String 컬럼으로 관리하던 Member를 테이블로 분리하고 MembeAction을 삭제합니다.

🏁 할 일

  • Member 엔티티 추가
  • MemberAction 삭제
  • API 변경

🫡 참고사항

테이블 분리 이유

  • 테이블에 Member의 이름이 중복된 데이터가 많이 발생
  • 이름 변경시 해당하는 모든 칼럼의 이름을 변경해야하는 문제 발생

엔티티 변경 사항

image
  • Member를 event와 name으로 생성하게 하여 memberAction은 Member와의 연관관계만 가질 수 있도록 변경합니다.
  • BillActionDetail도 Member을 참조하도록 변경합니다.

API 변경 사항

Member name의 String 값 대신 id를 주고 받을 수 있도록 API 엔드포인트 및 Request, Response 형태 변경이 필요합니다.

@Arachneee Arachneee added ⌨️ BE Backend 🚧 refactor refactoring labels Sep 9, 2024
@Arachneee Arachneee added this to the lev4 milestone Sep 9, 2024
@Arachneee Arachneee changed the title [BE] Member 엔티티 추가 [BE] Member 엔티티 추가 및 MemberAction 삭제 Sep 12, 2024
Arachneee pushed a commit that referenced this issue Sep 14, 2024
* test: BillController, BillService 지출 전체 조회 테스트 작성

* test: 참여자별 지출 고정 금액 조회 테스트 추가

* test: 지출 내역 추가 테스트 작성

* test: 지출 수정 테스트 작성

* refactor: docsTest responseFields 추가 및 액션이름 제거

* refactor: docsTest responseFields 추가 및 액션이름 제거
3Juhwan added a commit that referenced this issue Sep 19, 2024
* refactor: Member 엔티티 추가

Co-authored-by: Arachneee <jhg2819@naver.com>

* refactor: MemberAction 삭제

* test: 테스트 제

* test: 지출 고정 금액 수정, 삭제, 행사 조회, 정산 조회 테스트 추가

* test: �MemberAction 제거로 인한 Bill 도메인의 테스트 변경사항 반영 to feature/#546 (#557)

* test: BillController, BillService 지출 전체 조회 테스트 작성

* test: 참여자별 지출 고정 금액 조회 테스트 추가

* test: 지출 내역 추가 테스트 작성

* test: 지출 수정 테스트 작성

* refactor: docsTest responseFields 추가 및 액션이름 제거

* refactor: docsTest responseFields 추가 및 액션이름 제거

* test: 행사 도메인 테스트 변경

* feat: 행사 정보 수정 API 구현

* test: 테스트에 필요한 의존성 주입

* test: 행사 정보 업데이트 컨트롤러 테스트

* test: 행사 정보 업데이트 서비스 테스트

* test: 이벤트 도메인에 계좌 정보 설정 테스트

* test: 행사 업데이트 API 레독 테스트

* test: 행사 전체 참여 인원 조회 컨트롤러 테스트

* test: 행사 전체 참여 인원 조회 서비스 테스트

* test: 행사 전체 참여 인원 조회 API 레독 테스트

* test: 현재 참여 인원 API 관련 메서드 이동

* test: 현재 참여 인원 컨트롤러 테스트

* refactor: '현재 참여 인원 조회' 서비스 메서드명 수정

* test: '현재 참여 인원 조회' 레독 테스트 작성

* test: '현재 참여 인원 조회' 서비스 테스트

* test: 레독 테스트 수정

* test: 레독 테스트에 필수가 아닌 필드 값에 optional 설정

* refactor: 의미가 명확한 DTO를 사용하게 수정

* test: 현재 멤버 조회 기능 서비스 테스트

* test: 은행 정보를 enum으로 분리

* test: 새로운 API 구현, 테스트 작성 - part3 (#561)

* refactor: 참여자 생성 시 예외 형식 문서에 맞게 수정

* test: 참여자 추가 기능 테스트

* docs: 멤버 추가 예외 메시지 수정

* refactor: 멤버 삭제 로직 수정

* test: 멤버 삭제 테스트

* docs: 멤버 삭제 예외 수정

* test: 행사 참여자 정보 수정 컨트롤러 테스트

* docs: 참여자 정보 수정

* refactor: 참여자 정보 수정 검증 로직 변경

* test: 참여자 정보 수정 서비스 테스트

* test: 참여자 추가 테스트 수정

* refactor: 멤버 삭제 로직 수정

* refactor: updateMembers 메서드 삭제

* test: MemberService 테스트 수정

* refactor: 토큰 불일치 시 예외 던지도록 수정

* refactor: BillDetail 삭제 쿼리 한 번만 날리도록 수정

* test: 프로덕션 동작에 맞게 테스트 코드 수정

* refactor: 테스트에서만 사용하는 메서드 삭제

* refactor: 이름 중복 확인 로직 메서드로 분리

* [BE] 새로운 API 구현, 테스트 작성 - part4 (#566)

* refactor: 불필요 메소드 클래스 제거 및 액션 삭제

* refactor: Step 생성 로직 도메인으로 위임

* style: 주석 제거

* refactor: api 수정

---------

Co-authored-by: juha <khabh@naver.com>
Co-authored-by: Arachneee <jhg2819@naver.com>
Co-authored-by: kunsanglee <85242378+kunsanglee@users.noreply.github.com>
Co-authored-by: JUHA <84626225+khabh@users.noreply.github.com>
Co-authored-by: Arachne <66822642+Arachneee@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⌨️ BE Backend 🚧 refactor refactoring
Projects
Status: No status
Development

No branches or pull requests

4 participants