search

42trans / ft_transcendence

0 stars 0 forks source link

#280の後：本日プルリク２回目的なもの:endBtnにnulllのバグある #285

Closed uminomae closed 2 months ago

uminomae commented 2 months ago

e2e待ちで、前回の延長で行っています。 https://github.com/42trans/ft_transcendence/pull/280 前回プルリクがマージされていないので、変更に含まれています。

ここから２回分が実質の変更点です https://github.com/42trans/ft_transcendence/pull/285/commits/5cff30a1c169b5d175d33af821ab6c59b84c023a というか、それはコメントアウトしただけなので、 見てほしいコードはここだけ。２箇所にisvalid()メソッドを追加して毎行if(isValid)した https://github.com/42trans/ft_transcendence/pull/285/commits/fe356fb0ff5d7ee611bcd3878a4dc4efca9561b1

280 修正後、そこからブランチ切った

284

問題：最後のloopでnullが入る 理由：3D同様、spaのイベントでdispose()が走るため非同期でnullが先に入る 修正前状況：雑にリセットで対処されてたので要件的にはOK 修正後：毎行nullチェックを行い、console.warn()してreturnする処理。リセットはしない

uminomae commented 2 months ago

Ready for reviewですが、e2eと#280待ってからが良いでしょうか？そうでなければマージしてほしいです。 この後の作業もここから切りたいので。

ak0327 commented 2 months ago

まだ確認できていませんが、一旦両方マージしますね。 E2Eの修正は別ブランチ切ってやります

ak0327 commented 2 months ago

マージしました