ページングの挙動をいろいろ変えた

[stmtk1]

• 263 をやりました

• 266 をやりました

• 148 をやりました

• window.historyではなくreact routerのuseHistoryをつかうようにしました
• アカウントのランキングのテーブルを別コンポーネントに切り出しました

[stmtk1]

[stmtk1]

そこの実装をしようとすると実装がかなり重くなると思うので、別イシューを立てていいですか？

[ShopOne]

今はクエリパラメータを使わない実装になっていると思いますが、クエリパラメータを戻す実装にしても厳しいでしょうか？(ユーザー目線だと機能をそがれた形になるのであまり嬉しくなさそう)(誰も使ってない？)

[stmtk1]

使わない実装になってましたか。確認します

[stmtk1]

指摘された場所はきちんと動いてましたが、クリックしてもページが遷移しない実装になっていたので、それを修正しました。確認お願いします

[ShopOne]

まだ部分的に動いていなさそうな気がします？ PagingElementでparams.get('page');をしていると思うんですが、たぶんこれを以下に書き換えるといい気がします。 const paramPage = params.get('page') || '0';

あと、MainPageでかならずgetPastContestの0ページをセットしているのも良くなさそうで

 getPastContests(0).then((pastContestsInfo) => {
      setContestPageNum(
        Math.ceil(pastContestsInfo.allContestNum / CONTEST_IN_ONE_PAGE)
      );
    });

にすると動く気がします。 手元で軽く書いてちょっと確認しただけなので弱不安ですが…

[no-yan]

@ShopOne

まだ部分的に動いていなさそうな気がします？

どの部分の挙動が変わった/期待通りでないか、教えてもらえますか？ 具体的な項目があると修正しやすいと思います。

[ShopOne]

動作してほしい場合に動いてなかったのは以下の点です。 具体的な操作を書きますが、意味合いとしては伝わると思います。

• メインページの過去のコンテスト2ページ目をクリックし、次に名前をクリックして自分のページに移動、その後戻るボタンを押すとコンテストの2ページ目が開かれていてほしいが、1ページ目が開かれている(ページのボタンも2のまま)

[stmtk1]

修正しました

[vercel[bot]]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/shopone/shitforces/E7deFaqkaGLBWCpZXkb1iAdhVL6V
✅ Preview: Failed