[moin-review] 2021-11-19

Facts

progress-barbug修正

Findings

いいえ

Feelings

新しい送金プロセスでは、現在の送金の進捗を進捗バーで表すUIが追加される予定です.
問題は、異なる国、異なる初期選択の送金方法で、送金段階全体が異なることです.
例えば、日本では送金方法を選択できますが、アメリカの送金方法は口座送金に固定されています.
李京宇日本送金の場合、段階的に送金方法選択→受取人入力→edd順で行います.
アメリカ送金については、送金者入力→edd順に行います.
progress barの場合、ステップ全体の現在のステップの割合を%として計算する必要があります.
国ごとに、特定の送金方法を選択する際に、段階全体を変更します.
進捗バーはタイトルによって異なります.したがって、送金フォームのステータスを管理する親コンポーネントの子にタイトルを置くことはできません.
最上位管理ステータスから、reduxは送金フェーズのステータスを管理することを決定します.
通常、actionに基づいてステータスを更新し、コードの予測性を向上させるには、送金国/地域を選択する際にredoxに反映する必要があります.
送金を送る国の選択は、送金金額を入力する部分が乱れています.
送金伝票に入る際、国コードを受け取り、送金プロセスの更新を決定します.
どうせReduxで状態を管理する必要があるものもあるので、AhReduxで全体の状態管理をするのが正しいと思います.
このように、グローバルに不要な状態をreducxに入れると、限りなく
Reduxはますます大きくなっているので,状態管理部分の分離は正しい決定のようである.

// useRemitForm.ts 

const onSelectMethod = (method: Method) => {
    dispatch({ type: UPDATE_METHOD_TYPE, payload: method })
    reduxDispatch(updateRemitStep(getSteps({ method, countryCode })))
  }

useEffect(() => {
    reduxDispatch(updateRemitStep(getSteps({ countryCode })))
  }, [])