issues
search
mash-up-kr
/
piikii_Spring
피곤한 모임계획 끝, 키득키득 피키로 ( ͡~ ͜ʖ ͡°)
2
stars
0
forks
source link
feat: 투표 수정/생성/마감 구현
#49
Closed
thguss
closed
1 week ago
thguss
commented
2 weeks ago
이슈
for #39
변경 사항
수정
: 기존 room-update api를 활용했습니다. 추가로 Long이 아닌 UUID 타입의 roomId로 room을 찾도록 변경했습니다.
생성
: roomEntity 내에 voteDeadline이 존재하여 Room 관련 클래스에서 추가 구현했습니다.
마감
: 회원이 투표 요청에 접근하려고 할 때, voteDeadline과 현재 시간을 비교하여 유효하지 않으면 접근을 막도록 했습니다. 투표 요청에 필요한 서비스 메서드를 임의로 만들어 내부에 일부 구현했습니다.
스크린샷
부연 설명 (선생님들 의견이 궁금해욥)
투표 생성 조건이 voteDeadline 밖에 없어 room 관련 클래스 내에 관련 기능을 구현했는데, 괜찮을까요?-?
투표 생성 기능이 사실상 RoomEntity 정보를 업데이트하는 것인데, api를 별도로 빼는 게 좋은 방향인지 고민이 됩니다,,
별도로 뺄 경우, Room 도메인을 형성하기 위해서 클라이언트로부터 Room 필수 정보를 모두 받아야 하는데 좋은 방향일지도 고민되네용
방 정보 수정
,
투표 생성
,
투표 수정
기능을 하나의 api로 가져가는 것도 가능하긴 한데, 위 사항과 비교해서 무엇이 더 좋을 방향인지 마찬가지고 고민이 됩니당 🥹
추가 설명 06.22
우선 투표 open/close 기능 위주로 구현했습니다.
voteDeadline 값을 업데이트하는 기능은 room api 내에서 구현했습니다.
체크리스트
[x] Lint 적용 여부
[x] 빌드 성공 여부
[x] PR 제목은 포맷과 내용 둘 다 알맞게 작성되었는가
[x] PR에 대해 구체적으로 설명이 되어있는가
github-actions[bot]
commented
2 weeks ago
✅ 빌드 성공 ✅ 🚀 출격 준비 완료 👨🏻🚀
KimDoubleB
commented
2 weeks ago
출격준비
github-actions[bot]
commented
2 weeks ago
✅ 빌드 성공 ✅ 🚀 출격 준비 완료 👨🏻🚀
thguss
commented
2 weeks ago
출격준비
github-actions[bot]
commented
2 weeks ago
✅ 빌드 성공 ✅ 🚀 출격 준비 완료 👨🏻🚀
thguss
commented
1 week ago
출격준비
github-actions[bot]
commented
1 week ago
✅ 빌드 성공 ✅ 🚀 출격 준비 완료 👨🏻🚀
이슈
for #39
변경 사항
수정
: 기존 room-update api를 활용했습니다. 추가로 Long이 아닌 UUID 타입의 roomId로 room을 찾도록 변경했습니다.생성
: roomEntity 내에 voteDeadline이 존재하여 Room 관련 클래스에서 추가 구현했습니다.마감
: 회원이 투표 요청에 접근하려고 할 때, voteDeadline과 현재 시간을 비교하여 유효하지 않으면 접근을 막도록 했습니다. 투표 요청에 필요한 서비스 메서드를 임의로 만들어 내부에 일부 구현했습니다.스크린샷
부연 설명 (선생님들 의견이 궁금해욥)
방 정보 수정
,투표 생성
,투표 수정
기능을 하나의 api로 가져가는 것도 가능하긴 한데, 위 사항과 비교해서 무엇이 더 좋을 방향인지 마찬가지고 고민이 됩니당 🥹추가 설명 06.22
체크리스트