cloudnativedaysjp / dreamkast-ui

MIT License

7 stars 2 forks source link

Closed oshiro3 closed 3 years ago

oshiro3 commented 3 years ago

144

sessionStorage の使用で特に考慮すべき内容が無さそうだったので一旦実装してみました。 SSR等で考慮すべき内容や動作における問題等あれば報告いただけると助かります。

追記 sessionStorage のキーが重複していないことを確認しました

github-actions[bot] commented 3 years ago

oshiro3 commented 3 years ago

@maktak1995 ありがとうございます snapshot更新してyarn test:update コミットしてローカルではテスト通したんですけど、他どうすればCIに反映されますか？

maktak1995 commented 3 years ago

更新したsnapshotをpushすればCIに反映されるはずなんですけど、どうでしょうか？

oshiro3 commented 3 years ago

一応そうしているはずなんですけど。。もしかしたらブランチの名前変えて連続でプルリクオープンしたのが良くなかったのかも？

maktak1995 commented 3 years ago

CIが追従できていない的な感じですかね? ブランチ作り直すところからやったらいけるのかな...？

oshiro3 commented 3 years ago

もう1回スナップショット更新してpushしたんですけど、まだ落ちてますね。。なんだろう

maktak1995 commented 3 years ago

問題なさそうなんだけどなあ... めんどくさいとは思いますが、一度ブランチとPRを作り直してみるのがいいかもしれません

oshiro3 commented 3 years ago

ぼくのローカルの話なんですけど, 間違えて main にコミットしてそこからブランチ切ってPUSHしているのでグラフがおかしいとかはあるのかもしれないです. Actions の checkout フェーズで多分ベースにmergeして処理が走ってるっぽいので(さすがに関係なさそうですけど)

maktak1995 commented 3 years ago

だめだった...

oshiro3 commented 3 years ago

update した snapshot がそもそも間違っているとかってありえたりしそうですか？使用してる Node とか react のバージョンとかの関係で

maktak1995 commented 3 years ago

あれいまこのブランチをローカルに引っ張ってきてテスト回したんですがコケましたねちょっと手元でテストしてみてもらっていいですか？そのあと yarn test --updateSnapshot でsnapshotを更新してpushしてみていただきたく

maktak1995 commented 3 years ago

変わらなかったら試しに僕が手元でupdateしたやつをpushしてみていいですか？

oshiro3 commented 3 years ago

yarn install し直したら全コケが出ました。一回ぼくの方で push しますんでそれで直らなければお願いします！

maktak1995 commented 3 years ago

通りましたね 🎉

oshiro3 commented 3 years ago

通った... ローカルの依存関係が古かったのか、違うスナップショットが生成されていたようでした