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

feat: 입덕포인트 수정 및 통계 조회 구현 #346

Merged
merged 10 commits into from
Dec 28, 2023
Merged

Conversation

imdaxsz
Copy link
Contributor

@imdaxsz imdaxsz commented Dec 27, 2023

📝 개요

입덕포인트 수정 및 통계 조회 구현

  • 입덕포인트만 수정 시 (toast 메시지: 입덕포인트가 수정되었어요.)
test1.mp4

  • 입덕포인트와 리뷰 내용/스포여부 함께 수정 시 (toast 메시지: 리뷰가 수정되었어요.)
test2.mp4

🚀 변경사항

  • 스포일러 여부와 입덕포인트를 함께 설정 시 스포일러 여부가 attractionElements 배열에 함께 포함되는 버그 수정
  • 스포일러 여부 변경 시 스포일러 박스(SpoilerComment) 렌더링에 바로 반영되지 않는 버그 수정

🔗 관련 이슈

#303

➕ 기타

  • 개발 환경에서는 입덕포인트에 '작화' 포함 시 500 오류가 뜹니다. 배포 환경은 잘 동작합니다.
  • 입덕포인트 수정의 경우
기존: 캐릭터, 작화, 성우
수정 후: 캐릭터, 음악

위와 같이 변화가 있을 때 patch 요청에 생성과 같은 형태로 [캐릭터, 음악] 이 들어가게 되는데
현재 서버 쪽에서는 변화가 있는 요소 [작화, 성우, 음악] 만 받는 걸로 되어있습니다.
하니님께서 프론트 방식대로 수정해주신다고 하셔서 이대로 올립니다!

아직 서버 쪽 수정 반영이 되지 않아서 현재는 수정 요청에 기존 값에 포함된 요소를 포함하면 잘 동작하지 않습니다.
수정 반영 후에 다시 테스트가 필요합니다!


@imdaxsz imdaxsz added feature 새 기능 추가 fix 버그 등 수정 refactor 리팩토링 labels Dec 27, 2023
@imdaxsz imdaxsz self-assigned this Dec 27, 2023
@imdaxsz imdaxsz merged commit 9d63a6f into develop Dec 28, 2023
2 checks passed
@imdaxsz imdaxsz deleted the feature/303 branch December 28, 2023 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 새 기능 추가 fix 버그 등 수정 refactor 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants