cloudnativedaysjp / dreamkast-ui

MIT License
7 stars 2 forks source link

fix: Prevent blocking dashboard-page rendering when session-point QR code loaded but there are no live talk #412

Closed hrk091 closed 1 year ago

hrk091 commented 1 year ago

live talkが始まってない状態でセッション視聴によるスタンプ取得のQRコードを読み込むと、デッドロックになってしまう事象がでました。この状態になると、ローディングのグルグル画面が表示されたままdk-uiにアクセスできなくなります。 ユーザがこの事象にハマることは原則ありえないですが、スタッフはこのケースにハマることがありえてしまい、かつこうなるとsession storageのデータを消すしか対処がなくなるので、デッドロックにならないように修正しました。

やったことは、onAir talkがない分岐のところで、ただfail fastさせるだけではなく、loading画面のrenderingの解消とsession storageの破棄をする、という処理を追加しただけです。

github-actions[bot] commented 1 year ago

Review app