Skip to content

Feat/game redux#46

Merged
AndroidNetrunner merged 6 commits intodevelopfrom
feat/game-redux
Mar 23, 2022
Merged

Feat/game redux#46
AndroidNetrunner merged 6 commits intodevelopfrom
feat/game-redux

Conversation

@071yoon
Copy link
Copy Markdown
Collaborator

@071yoon 071yoon commented Mar 22, 2022

✅ PR check list

  • PR 제목: [feat/fix/refactor...] 작업 내용 한 줄 요약 (브랜치 이름) #이슈번호
  • commit message가 적절한지 확인해주세요.
  • 적절한 branch로 요청했는지 확인해주세요.
  • Assignees, Label을 붙여주세요.
  • 주의 사항과 관련해 꼭 확인해야 할 사람이 있다면 Reviewer로 등록해주세요.
  • PR이 승인된 경우 해당 브랜치는 삭제해 주세요!

📝 Changes

closes #43
closes #42
오류 빼고는 구현을 대충 다 끝냈습니다.

일단 I/O 에서 받고 보내는거 필요한만큼 IO Interface에 추가했고, 그 외로 이제 완전 redux 에서 놀 수있게, 원래 있던 임시로 만들어뒀던 정보들 다 삭제 및 DB 에서 직접 받아서 쓸 수 있게 설정 완료했습니다.

하지만 한번에 해결되면 코드가 아니죠? 예외처리 해야될거 개많습니다.

이따 이슈로 한꺼번에 올리겠지만 변경 및 주의사항에 적어 두겠습니다


👨🏻‍💻 Test Checklist

  • 이러한 테스트를 했어요!

📌 변경 및 주의 사항

우선 콘솔로그 찍히는거 이따 다 삭제해야됩니다.. 게임하다가 상대 코드를 미리 알아버리면 재미 없잖아요?

그리고 이상하게 게임이 끝났을 때 alert 가 안나옵니다. 아마도 수정본은 alert 후 navigation 으로 home 으로 보내주지 않을까 예상합니다

그리고 내가 제출하면, 제출안한 우리팀들은 전부 제출 버튼이 막히고 waiting 이 떠야되는데 지금 구현이 안됩니다.

아마 백엔드에서 이미 우리팀의 코드가 있다면 받지 않는걸로 설정하면서 WAITING (어떤 팀인지 넣어서) 이라는거를 broadcast 해주면, 같은 팀들이 그거 받아서 WAITING 뜨게 바뀌면 될거같네요...?

수정사항 있다면 알려주세요


@071yoon 071yoon added ✨enhancement New feature or request ❣️Front 프론트엔드 📡Back socket.io, data fetching labels Mar 22, 2022
@071yoon 071yoon self-assigned this Mar 22, 2022
@Seojunhwan
Copy link
Copy Markdown
Collaborator

일해라 백엔드

Copy link
Copy Markdown
Owner

@AndroidNetrunner AndroidNetrunner left a comment

Choose a reason for hiding this comment

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

👍

@AndroidNetrunner AndroidNetrunner merged commit 0b73400 into develop Mar 23, 2022
@AndroidNetrunner AndroidNetrunner deleted the feat/game-redux branch March 23, 2022 01:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

📡Back socket.io, data fetching ✨enhancement New feature or request ❣️Front 프론트엔드

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEATURE} 공통으로 사용하는 Interface 관리 [FEATURE] 전역 상태관리 Redux 도입

4 participants