Set packs on 2FA-related pages. Fixes #271.

Specifically, this commit:

- changes S::TFA::{Confirmations,RecoveryCodes}Controller to derive from
  S::BaseController, because this gives us the necessary actions and
  packs
- prepends set_pack to Auth::SessionsController's action chain so that
  it takes effect in time for render :two_factor
This commit is contained in:
David Yip 2017-12-20 03:15:54 -06:00
parent 6f11aa8383
commit bf1eb0912c
No known key found for this signature in database
GPG key ID: 7DA0036508FCC0CC
3 changed files with 3 additions and 11 deletions

View file

@ -8,8 +8,8 @@ class Auth::SessionsController < Devise::SessionsController
skip_before_action :require_no_authentication, only: [:create]
skip_before_action :check_suspension, only: [:destroy]
prepend_before_action :authenticate_with_two_factor, if: :two_factor_enabled?, only: [:create]
prepend_before_action :set_pack
before_action :set_instance_presenter, only: [:new]
before_action :set_pack
def create
super do |resource|

View file

@ -2,11 +2,7 @@
module Settings
module TwoFactorAuthentication
class ConfirmationsController < ApplicationController
layout 'admin'
before_action :authenticate_user!
class ConfirmationsController < BaseController
def new
prepare_two_factor_form
end

View file

@ -2,11 +2,7 @@
module Settings
module TwoFactorAuthentication
class RecoveryCodesController < ApplicationController
layout 'admin'
before_action :authenticate_user!
class RecoveryCodesController < BaseController
def create
@recovery_codes = current_user.generate_otp_backup_codes!
current_user.save!