Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ConsumerのCatalogView.vueに存在するローディング・スピナーの非表示制御をfinally句に移動する #1934

Open
1 task
kenjiyoshid-a opened this issue Jan 28, 2025 · 0 comments · May be fixed by #1959
Assignees
Labels
target: Dressca サンプルアプリケーションDresscaに関係がある 内部の改善 機能更新やドキュメント更新ではない改善
Milestone

Comments

@kenjiyoshid-a
Copy link
Contributor

概要

ConsumerのCatalogView.vueにおいて、ライフサイクルフックであるonMountedメソッド内でローディング・スピナーの表示非表示を制御している。現状の実装では、onMountedメソッドのはじめにstate.showLoading = trueとし、try-catch文を挟んで、state.showLoading = falseとしている。
グローバルエラーハンドラーが存在するため、catch文の中で例外が発生したとしても、エラーページに遷移されるが、finally句にstate.showLoading = falseを記述した方が安全であるため修正を行う。

詳細 / 機能詳細(オプション)

なし。

完了条件

  • ConsumerのCatalogView.vueにおいて、ローディング・スピナーの非表示がfinally句内で実装されている。
@kenjiyoshid-a kenjiyoshid-a added not triaged 精緻化、分析がされていない target: Dressca サンプルアプリケーションDresscaに関係がある and removed not triaged 精緻化、分析がされていない labels Jan 28, 2025
@kenjiyoshid-a kenjiyoshid-a self-assigned this Feb 5, 2025
@kenjiyoshid-a kenjiyoshid-a added this to the v1.1.0 milestone Feb 5, 2025
@kenjiyoshid-a kenjiyoshid-a added the 内部の改善 機能更新やドキュメント更新ではない改善 label Feb 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
target: Dressca サンプルアプリケーションDresscaに関係がある 内部の改善 機能更新やドキュメント更新ではない改善
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant