AlmaLinux / AlmaLinux/build-system

Review routes and ensure that there are no more navigation warnings around calling next more than once

オープン
#435 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
good first issue ui
主要言語
言語のデータがありません
スター
32
フォーク
11
PR マージ指標
30日以内にマージされた PR はありません

説明

As unveiled in the [following issue](https://github.com/AlmaLinux/build-system/issues/430) and resolved in [this PR](https://github.com/AlmaLinux/albs-frontend/pull/588), there might be some navigation in other places due to use of calling `next()` as part of promise handling in the routing process.

We should review albs-frontend's [routes.js](https://github.com/AlmaLinux/albs-frontend/blob/master/src/router/routes.js) and ensure that:
* We stop calling next, and pass it as a callback in the promise resolution process
* Several store/state dispatchers in `beforeEnter` should be wrapped into a Promise.all

After the update, we need to check that the navigation works as expected and that there are no more navigation warnings around.

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。