search

ShopOne / Shitforces

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

順位表の時間表示を一部修正　 #182

Closed no-yan closed 3 years ago

no-yan commented 3 years ago

close #170 コンテストページ得点欄の時間表示のフォーマットを変えました。 コミットメッセージで間違えてますが、フォーマットは分分：秒秒ですね。中身は大丈夫です。

周辺の型定義を追加、コンポーネントを切り出しました。 reactはコンポーネントをなるべく小さく分けて、レイアウト（ビュー）部分とロジック部分を独立させることがベストプラクティスのようです。コンポーネントを細かく分けることはパフォーマンス戦略としても優れていますし、独立性を高めることはメンテの容易さにも繋がります。

@rdrgn PRのリファクタリング部分で#133の作業とコンフリクトしている部分があれば、ご自身のファイルの方を優先してくださって大丈夫です！コンフリクト解決は面倒だと思うので……

reminjp commented 3 years ago

PRのリファクタリング部分で#133の作業とコンフリクトしている部分があれば、ご自身のファイルの方を優先してくださって大丈夫です！コンフリクト解決は面倒だと思うので……

お気遣いありがとうございます。問題の #133 は重すぎて半ばで放置しておりましたので、無視して大丈夫です。むしろリファクタリングを進めていただけてありがたいです。

no-yan commented 3 years ago

丁寧なレビューありがとうございます。二度目の指摘は勇気がいりますが、受ける側としては率直に言ってもらえることでとてもやりやすく感じます。 （こちらからはreviewerを変更することができないので、 @ShopOne さんの負担が（雑務的な方向性で）増えそうなのを少し気にしています。）

PointAndAcTime関数はprobListでしか参照されないコンポーネントなので削除しました。

ShopOne commented 3 years ago

こちらも内心ビクビクしながら Review を書いているので、そう行って頂けると精神的にとても助かります。ありがとうございます。 僕の作業量に関しては気にせず、どしどしプルリクエストを出してもらってOKです！ (忙しい時は反応遅れちゃうかもですが) commit の内容については大丈夫そうです！ 再レビュー状態になっていないので、一応まだ merge は待機中です。 ついでのお願いですが、 conflict しているのを直して頂けると嬉しいです。(見た感じ probElement 消すだけっぽいですが)