diff --git a/app/assets/stylesheets/components/_icon.scss b/app/assets/stylesheets/components/_icon.scss index 708d8447270..661fd037fb0 100644 --- a/app/assets/stylesheets/components/_icon.scss +++ b/app/assets/stylesheets/components/_icon.scss @@ -20,22 +20,3 @@ $icon-min-padding: 2px; margin-right: #{0.5rem - px-to-rem($icon-min-padding)}; } } - -.ico-absolute { - background-repeat: no-repeat; - background-size: $h4; - position: relative; - - &-success { - &::before { - background-image: url('/alert/success.svg'); - content: ''; - display: block; - height: $h4; - left: units(neg-4); - position: absolute; - top: (lh('body', $theme-body-line-height) - $h4) * 0.5; - width: $h4; - } - } -} diff --git a/app/controllers/users/verify_password_controller.rb b/app/controllers/users/verify_password_controller.rb index 2e656413473..fd006b6a8f7 100644 --- a/app/controllers/users/verify_password_controller.rb +++ b/app/controllers/users/verify_password_controller.rb @@ -6,12 +6,9 @@ class VerifyPasswordController < ApplicationController before_action :confirm_password_reset_profile before_action :confirm_personal_key - def new - @decrypted_pii = decrypted_pii - end + def new; end def update - @decrypted_pii = decrypted_pii result = verify_password_form.submit irs_attempts_api_tracker.logged_in_profile_change_reauthentication_submitted( @@ -32,13 +29,6 @@ def confirm_personal_key redirect_to root_url end - # rubocop:disable Naming/MemoizedInstanceVariableName - # @return [Pii::Attributes, nil] - def decrypted_pii - @_decrypted_pii ||= reactivate_account_session.decrypted_pii - end - # rubocop:enable Naming/MemoizedInstanceVariableName - def handle_success(result) flash[:personal_key] = result.extra[:personal_key] irs_attempts_api_tracker.idv_personal_key_generated @@ -50,7 +40,7 @@ def verify_password_form VerifyPasswordForm.new( user: current_user, password: params.require(:user).permit(:password)[:password], - decrypted_pii: decrypted_pii, + decrypted_pii: reactivate_account_session.decrypted_pii, ) end end diff --git a/app/views/shared/_pii_review.html.erb b/app/views/shared/_pii_review.html.erb deleted file mode 100644 index 37c5c82f5a5..00000000000 --- a/app/views/shared/_pii_review.html.erb +++ /dev/null @@ -1,42 +0,0 @@ -
-
- <%= t('idv.review.full_name') %> -
- -
- <%= pii[:first_name] %> <%= pii[:last_name] %> -
- -
- <%= t('idv.review.mailing_address') %> -
- -
- <%= render 'shared/address', address: pii %> -
- -
- <%= t('idv.review.dob') %> -
- -
- <%= DateParser.parse_legacy(pii[:dob]).to_formatted_s(:long) %> -
- -
- <%= t('idv.review.ssn') %> -
- -
- <%= pii[:ssn] %> -
- - <% if phone %> -
- <%= t('idv.messages.phone.phone_of_record') %> -
-
- <%= PhoneFormatter.format(phone) %> -
- <% end %> -
diff --git a/app/views/users/verify_password/new.html.erb b/app/views/users/verify_password/new.html.erb index 4062087225e..a05a0abd77f 100644 --- a/app/views/users/verify_password/new.html.erb +++ b/app/views/users/verify_password/new.html.erb @@ -23,10 +23,3 @@ ) %> <%= f.submit t('forms.buttons.continue'), class: 'margin-top-5' %> <% end %> - -
- <%= render AccordionComponent.new do |c| %> - <% c.with_header { t('idv.messages.review.intro') } %> - <%= render 'shared/pii_review', pii: @decrypted_pii, phone: @decrypted_pii[:phone] %> - <% end %> -
diff --git a/config/locales/idv/en.yml b/config/locales/idv/en.yml index ca19f31e4de..23976eacca7 100644 --- a/config/locales/idv/en.yml +++ b/config/locales/idv/en.yml @@ -213,14 +213,12 @@ en: alert_html: 'Enter a phone number that is:' description: We’ll check this number with records and send you a one-time code. This is to help verify your identity. - phone_of_record: Phone of record rules: - Based in the United States (including U.S. territories) - Your primary number (the one you use the most often) return_to_profile: '‹ Return to your %{app_name} profile' review: gpo_pending: We’ll send your letter once you re-enter your password. - intro: Your verified information message: '%{app_name} will encrypt your information with your password. This means that your information is secure and only you will be able to access or change it.' @@ -235,11 +233,6 @@ en: review_message: When you re-enter your password, %{app_name} will protect the information you’ve given us, so that only you can access it. verifying: Verifying… - review: - dob: Date of birth - full_name: Full name - mailing_address: Mailing address - ssn: Social Security number (SSN) titles: activated: Your identity has already been verified come_back_later: Your letter is on the way diff --git a/config/locales/idv/es.yml b/config/locales/idv/es.yml index f2368f24d79..8de261b44d7 100644 --- a/config/locales/idv/es.yml +++ b/config/locales/idv/es.yml @@ -228,14 +228,12 @@ es: alert_html: 'Introduzca un número de teléfono que sea:' description: Comprobaremos este número con los registros y le enviaremos un código único. Esto es para ayudar a verificar su identidad. - phone_of_record: Teléfono del registro rules: - Con base en Estados Unidos (incluidos los territorios de EE.UU.) - Su número principal (el que utiliza con más frecuencia) return_to_profile: '‹ Volver a tu perfil de %{app_name}' review: gpo_pending: Enviaremos tu carta una vez que hayas ingresado tu contraseña. - intro: Su información verificada message: '%{app_name} encriptará tu información con tu contraseña. Esto significa que tu información estará segura y solo tú podrás consultarla o modificarla.' @@ -251,11 +249,6 @@ es: review_message: Cuando vuelva a ingresar su contraseña, %{app_name} cifrará sus datos para asegurarse de que nadie más pueda acceder a ellos. verifying: Verificando… - review: - dob: Fecha de nacimiento - full_name: Nombre completo - mailing_address: Dirección postal - ssn: Número de Seguro Social (SSN, sigla en inglés) titles: activated: Ya se verificó tu identidad. come_back_later: Su carta está en camino diff --git a/config/locales/idv/fr.yml b/config/locales/idv/fr.yml index 5c7316cb18c..32ee3c3b972 100644 --- a/config/locales/idv/fr.yml +++ b/config/locales/idv/fr.yml @@ -243,7 +243,6 @@ fr: alert_html: 'Entrez un numéro de téléphone qui est :' description: Nous vérifierons ce numéro dans nos archives et vous enverrons un code à usage unique. Ceci est pour aider à vérifier votre identité. - phone_of_record: numéro de téléphone enregistré rules: - Basé aux Etats-Unis (y compris les territoires américains) - Votre numéro principal (celui que vous utilisez le plus souvent) @@ -251,7 +250,6 @@ fr: review: gpo_pending: Nous vous enverrons votre lettre une fois que vous aurez réintroduit votre mot de passe. - intro: Vos informations vérifiées message: '%{app_name} crypte vos informations avec votre mot de passe. Cela signifie que vos informations sont sécurisées et que vous seul pourrez y accéder ou les modifier.' @@ -268,11 +266,6 @@ fr: review_message: Lorsque vous entrez à nouveau votre mot de passe, %{app_name} crypte vos données pour vous assurer que personne ne peut y accéder. verifying: Vérification… - review: - dob: Date de naissance - full_name: Nom complet - mailing_address: Adresse postale - ssn: Numéro de sécurité sociale (SSN) titles: activated: Votre identité a déjà été vérifiée come_back_later: Votre lettre est en route diff --git a/spec/controllers/users/verify_password_controller_spec.rb b/spec/controllers/users/verify_password_controller_spec.rb index 34c1db19235..55604aa76a5 100644 --- a/spec/controllers/users/verify_password_controller_spec.rb +++ b/spec/controllers/users/verify_password_controller_spec.rb @@ -91,14 +91,12 @@ end context 'without valid password' do - let(:pii) { { dob: Time.zone.today } } let(:response_bad) { FormResponse.new(success: false, errors: {}) } render_views before do allow(form).to receive(:submit).and_return(response_bad) - allow(controller).to receive(:decrypted_pii).and_return(pii) put :update, params: user_params end