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

[GGFE-39] 게임 결과 입력 #809

Merged

Conversation

yoouyeon
Copy link
Member

@yoouyeon yoouyeon commented Jun 2, 2023

📌 개요

  • 게임 후 결과 입력에 대한 api 변경사항 적용

💻 작업사항

  • 변경이 덜 되었던 api url 변경사항과 MatchMode 타입 변경사항 적용했습니다.
  • modalMode 타입을 MatchMode 타입으로 사용하면 'BOTH' 값이 없는 것에 문제가 있어서 따로 리터럴 타입으로 만들었습니다.
  • modalMode 중 confirm modal을 사용하는 부분이 없는 것 같은데, 모든 경우를 테스트해본게 아니라 일단 주석으로 남겨두었습니다.

❗️ statChangeModal에 표시되는 값이 이상한 것은 api response 가 이상했던 것이라 수정된 api 배포되면 확인해보고 수정사항 있을 시에 다시 PR 올리겠습니다!!
확인 완료

@yoouyeon yoouyeon self-assigned this Jun 2, 2023
Copy link
Contributor

@42sungwook 42sungwook left a comment

Choose a reason for hiding this comment

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

문제는 없어보이는데 백에서 api수정이후 다시 확인해야 할 부분으로 보여서 이슈 닫아놓을게요!

Comment on lines +20 to +25
// NOTE : 이전부터 사용이 안되던 부분인 것 같아서 일단 주석으로 남겨두었습니다.
// CONFIRM: {
// before: '경기 결과!',
// after: '경기 결과!',
// explains: '이미 입력된 경기입니다. 점수를 확인하세요!\n',
// },
Copy link
Contributor

Choose a reason for hiding this comment

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

이 부분은 상대방이 먼저 점수 입력했을 때 뜨는 메세지 아닌가요?
본적이 있는 문구인 것 같은데 다시 확인해봐야겠네요

Copy link
Member Author

Choose a reason for hiding this comment

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

'confirm' 을 검색했을 때 이 단어 자체가 사용되는 부분이 이 부분밖에 없긴 한데요... 🤔
제가 알기론 상대방이 점수를 입력했을 때에는 isScoreExist 값을 확인해서 다시입력 , 제출하기 버튼이 뜨는게 아니라 게임 종료 버튼이 바로 뜨는 방식이라 따로 confirm modal(?)을 띄우는 부분이 없을 것 같기도 해요.

Copy link
Member Author

Choose a reason for hiding this comment

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

다시 확인해보니까 기존과는 달리 현재 변경된 api로는 한명이라도 점수를 입력하면 경기에 참여한 모든 사람들의 상태가 게임 종료로 떠서 confirm modal을 아예 띄울 수가 없네요,,, 🥹 이부분은 users/live api에서 다시 로직을 확인해봐야 할 것 같습니다...😵‍💫

@42sungwook 42sungwook closed this Jun 4, 2023
@yoouyeon
Copy link
Member Author

yoouyeon commented Jun 5, 2023

stat change api 수정사항 확인했고 잘 동작해서 다시 PR 오픈하겠습니다!

@yoouyeon yoouyeon reopened this Jun 5, 2023
@42sungwook 42sungwook merged commit 45af8db into GGFE-00/Feat/api-refactoring Jun 8, 2023
@yoouyeon yoouyeon deleted the GGFE-39-게임-결과-입력 branch June 14, 2023 09:01
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.

2 participants