内容
IntegrationsSection.vue の importStarted / importError は、TanStack Query の mutation が既に持っている状態の二重管理です。importMutation.isSuccess / isError に置き換えると ref 2 本と try/catch が消え、再クリック時のメッセージ消去も mutation のリセットで自動的に得られます(レビュー時に置き換えて全テスト通過・-5 行を確認済み)。
見送った理由
同ファイルのインストール / 連携解除、および LabelsSection / CustomFieldsSection / WorkflowStatusesEditor が ref + try/catch で揃っているため、取り込みだけ書き方を変えると一貫性が崩れます。やるならファイル単位・セクション単位でまとめて寄せるのが妥当です。
機能上の問題はありません。#590 のレビューからの積み残しです。
内容
IntegrationsSection.vueのimportStarted/importErrorは、TanStack Query の mutation が既に持っている状態の二重管理です。importMutation.isSuccess/isErrorに置き換えると ref 2 本と try/catch が消え、再クリック時のメッセージ消去も mutation のリセットで自動的に得られます(レビュー時に置き換えて全テスト通過・-5 行を確認済み)。見送った理由
同ファイルのインストール / 連携解除、および
LabelsSection/CustomFieldsSection/WorkflowStatusesEditorが ref + try/catch で揃っているため、取り込みだけ書き方を変えると一貫性が崩れます。やるならファイル単位・セクション単位でまとめて寄せるのが妥当です。機能上の問題はありません。#590 のレビューからの積み残しです。