search

ShopOne / Shitforces

くそなぞなぞコンテストサイト
shitforces.vercel.app
MIT License
27 stars 13 forks source link

Disable to watch user history exept your own#244 #247

Closed no-yan closed 2 years ago

no-yan commented 2 years ago

close #244

このPRは以下の変更を含みます

  1. ユーザーページの遷移に関するbug fix : #244
  2. eslintの設定の更新
  3. リファクタリングとスタイリング

遷移に関する問題

ユーザーデータを取得する前にコンポーネントが描画され、ユーザーデータが更新されても再描画されていませんでした。データ取得の際にまず初期化を挟むことで更新を強制しました。

eslint

設定を更新しました。これまでimport順のエラーが出た際は手動でなおしていましたが、今後はautofixされます。typescriptのエラーについてもチェックされるよう追加しました。 一つの関数が100行以上になるとき、警告を出すようにしました。不必要であれば教えて下さい。 eslintが関係ないファイルを検査していたので、eslintignoreに無視するファイルを定義しました。これによりチェックが高速化します。vscodeなど常にeslintのエラーをチェックしている場合、だいぶ軽量化するはずです。

リファクタリング(最後のコミット)

かなり我を出してリファクタリングしました。 コンポーネントを分割し、一つのコンポーネントが一つの関数からなるよう変更しました。 また、一関数内でUIの記述が離散的だったので、一つの場所に集めました。UIを一つの場所に集約することでコンポーネントの内容がわかりやすくなるのではないかと考えています。

自分の意見が多くはいったPRなので、別の意見もあるだろうことは承知しています。フィードバックがほしいです。よろしくおねがいします。

ShopOne commented 2 years ago

おお… これは凄いですね ありがとうございます！ 確認させて頂きます。