-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
refactor: 拡張機能インストールのページの一部をコンポーネントとして分離 #14654
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #14654 +/- ##
============================================
- Coverage 41.74% 19.50% -22.25%
============================================
Files 1549 718 -831
Lines 196555 101655 -94900
Branches 2767 989 -1778
============================================
- Hits 82055 19826 -62229
+ Misses 113939 81281 -32658
+ Partials 561 548 -13 ☔ View full report in Codecov by Sentry. |
<!-- TODO: ハッシュ検証に失敗した場合の表示も作る --> | ||
<!-- その場合、hashVerifiedがfalseなら失敗表示、undefinedなら表示なしで分けたい --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ハッシュ検証のエラー画面出し分けはinstall-extensions.vueでやってるので、失敗した場合のハンドリングをここでやる必要はないかも
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
このコンポーネント分離は他の場所からも使用できるようにすることを念頭に置いています。
install-extensions.vue以外から使われる可能性があるので、場合によってはここでハンドリングすることもあると思います。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ならエラー画面もろとも引っ越す必要がありそう(ハッシュ検証に失敗した場合はそもそもインストールさせないので)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
すみません、色々混乱していました。
よく考えたらこのコンポーネントは表示専用に作っているのでエラーハンドリングを行わせる予定は最初から無かったですし、懸念事項は「props.hashVerifiedがboolean型なのを見て、falseにすれば失敗画面が出ると勘違いする人が出る」ことだったので単にコメントで注釈を書けばいいですね。その方向で修正します。
もしくはもっと柔軟にするべくハッシュ計算とかの部分を |
そのようにしました。 |
これテストするのどうしたら良いかしら |
storybookで十分な気もしますが… |
👍🏻 |
What
「外部サイトから(拡張機能を)インストール」のページの一部をコンポーネント化します。
Why
将来プラグインやテーマをインストールする窓口を増やす時にUIを流用可能にするため。
#12986 など
Additional info (optional)
Checklist