招待URL作成時に研修生を支払い方法別に選択可能にした・研修生の場合は会員登録時の支払い方法の指定を必須にした #8013

・これまで管理者用の項目だった「請求書払いの設定」のUIを削除し、支払い方法（請求書orクレジットカード）を選択するためのチェックボックスを一般ユーザー向けの項目として新たに作成した。併せて、登録フォームに遷移する前の招待URL作成ページのロール欄でどれを選択したかによって、登録フォームが表示された初期状態でどちらの支払い方法にチェックが入っているか（または入っていないか）を振り分けるように修正した。

…装・チェックボックスのパーシャルの名称を変更・コントローラーで研修生の支払い方法に関する情報を@pay_by_invoiceと@pay_by_credit_cardに格納していたが、params[:role]の参照で事足りるため、これら二つのインスタンス変数は廃止した。・「研修生（支払い方法を選択）」を選んだ場合は参加登録フォーム上でチェックボックスのチェックを変更できることから、カード番号入力フォームの表示/非表示を切り替えができる必要があったため実装した。・チェックボックスのパーシャルは既存のUIを流用したため名称が_invoice_paymentのままになっていたが、実態に即していないため変更した。

…wアクション内の条件分岐の書き方を変更・招待URL作成ページで「研修生（請求書払い）」を選択した場合は登録フォームの「請求書払い」のチェックボックスはtrueでなくてはならないため、そのバリデーションを追加した。・ビューでparamsを直接呼び出すとURLに含まれるパラメーターに変更があった場合の対応が煩雑になる恐れがあるため、ビューで必要な情報はコントローラーでインスタンス変数に代入して使用することとした。・UsersControllerのnewアクションではparams[:role]の値によって分岐する処理をif文で記述していたが、case文で記述した方が可読性が高いと判断したため変更した。

・クレジットカードのチェックボックスのDOM要素を取得する際getElementsByNameを使用していたが、対象の要素が見つからない場合でも空のNodeListが返ってしまい、条件分岐が正しく動作しないため、要素が見つからない場合にnullを返すquerySelectorに置き換えた。

・招待URL作成ページで選択された支払い方法のチェックボックスは登録フォームが表示された時点でチェック済みになっているが、これまではそのチェックを外すことができてしまっていた。そのため、誤操作でチェックを外してしまい無駄にバリデーションエラーになる可能性も考えられたため、ユーザーがチェックを外せないようにした。

・バリデーションで引っかかった場合はrenderで再度newテンプレートが呼び出されるが、招待URL作成ページからの遷移でない場合は必要なパラメータがURLに含まれておらず、チェックボックスが操作不能になる不具合が出ていたため修正した。

・招待URL作成ページにて「研修生（支払い方法を選択）」が選択された場合、登録ページのチェックボックスの挙動が、 (1)「請求書払い」と「クレジットカード払い」のいずれか一方しかチェックできない、 (2)「クレジットカード払い」が選択されている時のみカード情報の入力フォームを表示する、となるように実装した。それに伴い、カード情報の入力フォームの表示/非表示の制御と2種類のチェックボックスの状態制御を同時に行う必要があったため、card-form-display-switcher.jsで行っていた処理をpayment-methods-check-boxes.jsに統合した。

・DOMツリーにクレジットカード情報の入力フォームが存在すると、カード情報が空欄の場合や有効な番号でない場合にフォーム全体の送信を止める機能が実装されているが、招待URL作成ページで「研修生（支払い方法を選択）」を選んだ場合、クレジットカードフォームが空欄の状態であってもフォームが送信されなければならない（請求書払いを選択している）状況があり得ることになるため、「クレジットカード払い」のチェックボックスにチェックがない場合には当該バリデーションを無効化するように変更した。

・「研修生(クレジットカード払い)」を選択した場合の登録フォームでStripeのカード番号の入力欄が正しくマウントされていなかったため修正。

@user

・users#newアクションの中でuser_paramsを呼び出してしまっていたことで、/users/newなどのパラメーターを持たないか不十分なURLではParameterMissingエラーが発生し、アクセスできない状況となっていたため修正した。併せて、変数@ROLEは廃止し、@userのパラメーターとして扱うことでパーシャルに渡す変数の削減も行った。

・支払い方法を選択できる場合の登録フォームにあるselectableCreditCardCheckBoxにチェックが入っているかどうかがフォームをそのまま送信するかクレジットカードのバリデーションを通すかの条件となっており、請求書払いが確定している登録フォーム上では存在しない要素のプロパティを呼び出すことになってしまいエラーとなっていたため修正した。

・クレジットカードのバリデーションを作動させる条件にcheckedCreditCardCheckBoxが含まれておらず正常に動作していなかったため修正した。

・メンター用の登録フォームの場合は「FBCメンター参加登録」と表示されていなければならないところ、一般受講生と同じ「FBC参加登録」となってしまっていたため修正した。

・クレジットカード情報の入力を伴う他のテストに倣ってVCRでログをとるように修正した。

・引数checkBoxesの要素である二つのチェックボックスのうち少なくとも一つは必ずnullが渡ってくるためif文で存在確認をしていたが、オプショナルチェーン演算子(?.)を使用するほうがより簡潔に書けるため変更した。

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

招待URL作成時に研修生を支払い方法別に選択可能にした・研修生の場合は会員登録時の支払い方法の指定を必須にした #8013

招待URL作成時に研修生を支払い方法別に選択可能にした・研修生の場合は会員登録時の支払い方法の指定を必須にした #8013

Commits on Aug 21, 2024

招待URL作成時に研修生を支払い方法別に選択可能にした・研修生の場合は会員登録時の支払い方法の指定を必須にした #8013

Are you sure you want to change the base?

招待URL作成時に研修生を支払い方法別に選択可能にした・研修生の場合は会員登録時の支払い方法の指定を必須にした #8013

Commits on Aug 21, 2024