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

Impl Entity API (MEE013~027) #1298

Open
wants to merge 12 commits into
base: dev
Choose a base branch
from

Conversation

ChaeyeonAhn
Copy link
Contributor

@ChaeyeonAhn ChaeyeonAhn commented Jan 5, 2025

요약 *

It closes #1297

  • MEE013 - 027 구현 완료했습니다

  • 테스트는 아직 못 해봤어요..! 26일까지 해보겠습니다!

Update 01/27

MEE024-MEE026을 그냥 Vote에 대해 모든 정보를 가져오는 API (24번)하나로 수정했습니다.
  • 그에 따라 비는 번호들은 그냥 비어 있도록 놔두었어요

  • 13번부터 27번까지 모두 테스트 완료했습니다!

  • 13: Content 생성, Mapping 생성

  • 14: Content 내용 수정

  • 15: Content 삭제, Mapping 삭제 (모두 soft delete)

  • 16: Vote 생성, 매핑 생성, 선택지 생성

  • 17: Vote에 대한 유저의 투표 내역 생성

  • 18: Vote의 제목, 설명 수정

  • 19: Vote의 선택지 수정 (선택지 항목 개수가 늘어날 수도 있어서, 모두 삭제 후 재생성하는 로직)

  • 20: Vote에 대한 유저의 투표 내역 수정

  • 21: Entities 순서 변경

  • 22: Vote 삭제, 매핑 삭제, 선택지 모두 삭제, 유저의 투표 내역들 삭제 (모두 soft delete)

  • 23: Vote에 대한 유저의 투표 내역 삭제

  • 24: Vote에 대한 모든 정보 가져오기 (제목, 설명, 선택지들, 유저가 선택한 것, 최종결과 (표 집계내역))

  • 27: 특정 미팅 특정 아젠다의 모든 엔티티 가져오기

스크린샷

이후 Task *

  • 저번에 말씀하신 유저들이 투표율을 보는 API 짜기
  • SSE 관련 찾아보기

@ChaeyeonAhn ChaeyeonAhn added enhancement New feature or request back-end labels Jan 5, 2025
@ChaeyeonAhn ChaeyeonAhn self-assigned this Jan 5, 2025
@ChaeyeonAhn ChaeyeonAhn linked an issue Jan 5, 2025 that may be closed by this pull request
2 tasks
Copy link
Contributor

@Gerbera3090 Gerbera3090 left a comment

Choose a reason for hiding this comment

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

전체적으로 보이는 건

  1. 모듈 구조를 entity로 묶고 repository를 분리하기
  2. patch랑 put을 저는 반대 의미 (put이 데이터 밀어넣기)로 알고 있는데, 여긴 다 patch로 되어 있어서 put으로 수정하기
  3. patch든 put이든 httpstatus created로 되어 있는 부분들 ok로 변경하기
  4. 18,19,20 번 url 같은거 뒤에 엔티티 더 붙여서 수정하기
  5. 17번도 url 마지막에 result 붙이면 더 좋을 듯?
    양 참 많은데 고생 많았어요~~

Copy link
Contributor

Choose a reason for hiding this comment

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

meetingAgendaEntityType은 Enum이 아닌가? enum이라면 enum 테이블과 네이티브 이넘으로 만들면 좋을 것 같아요

Copy link
Contributor

Choose a reason for hiding this comment

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

테이블에 안만들기로 한건가? 그래도 native enum은 있어야 할 것 같은데...

Copy link
Contributor

Choose a reason for hiding this comment

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

status created 수정 필요?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

그리고 이넘 테이블은 아는 바가 없어요.. 설계 당시에는 그냥 이넘만 쓰기로 했습니다

Copy link
Contributor

Choose a reason for hiding this comment

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

지금 repository 를 entity에서 공유하고, 나머지 vote랑 content를 서브모듈로 둔 것 같은데, 구조가 반대가 되어야 하지 않을까 싶습니다. Entity 모듈, 서비스, 컨트롤러를 쓰면서 각 서브 엔티티(vote, content) 별로 repository를 두는 방식으로요

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Entity 모듈, 서비스, 컨트롤러를 쓰면서 각 서브 엔티티(vote, content) 별로 repository를 두는 방식으로요

이거의 경우에는 vote, content 모듈을 아예 없애라는 말씀이신가요?
아니면 vote content 모두 모듈 서비스 컨트롤러에 repository까지 두라는 말씀이신가요

@ChaeyeonAhn ChaeyeonAhn marked this pull request as ready for review January 20, 2025 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
back-end enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Impl Entity API MEE 013 to 027
2 participants