Skip to content

Feature/ranking - #29

Open
mrskiro wants to merge 58 commits into
developfrom
feature/Ranking
Open

Feature/ranking#29
mrskiro wants to merge 58 commits into
developfrom
feature/Ranking

Conversation

@mrskiro

@mrskiro mrskiro commented May 29, 2020

Copy link
Copy Markdown
Contributor

【概要】
ランキングページの追加及びデータの取得、追加、絞り込み、ソート

【変更内容】
firestoreと連携しました。
公開することを意識して、チュートリアルと画面上部のヘッダーを追加しました。

【詳細】
ランキングページ→react-tableの使用

https://minesweeeeeper.web.app/
なんとか動いていると思います。
よろしくおねがいします!

@mrskiro
mrskiro requested a review from so99ynoodles May 29, 2020 09:16
@mrskiro

mrskiro commented May 29, 2020

Copy link
Copy Markdown
Contributor Author

soundも流せるようになりました。
お時間ある時に確認お願いします!

@so99ynoodles

Copy link
Copy Markdown
Contributor

yarn start して確認しようとしたら、typescript側でエラーが出る…
image

@so99ynoodles

This comment has been minimized.

@mrskiro

This comment has been minimized.

@so99ynoodles

This comment has been minimized.

@mrskiro

mrskiro commented Jun 1, 2020

Copy link
Copy Markdown
Contributor Author

わかりました!やってみます!

@mrskiro

mrskiro commented Jun 1, 2020

Copy link
Copy Markdown
Contributor Author

すべて修正できました!確認お願いします!

デプロイもしています
https://minesweeeeeper.web.app/

@so99ynoodles so99ynoodles left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

気になって点としては、rankingに行った場合の挙動かな。

  1. 音楽が途切れてしまう
  2. 音楽を無効にしてrankingに行って戻った時にまた流れる
  3. そのまま戻ったらゲームが再開されるけど、rankingの中でlevelを変えてから戻ると変えたlevelでやり直しになるから、挙動が統一されてないと感じちゃう

あと、前にslackにも投げたけど、クリアしてないのにクリアした扱いにされたケースがあるので、クリアのロジックにバグがある可能性があるね、、
image

Comment thread tsconfig.json Outdated
Comment thread src/components/InputForm.tsx Outdated
Comment thread src/containers/Ranking.tsx Outdated
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