diff --git a/src/account-settings/AccountSettingsPage.jsx b/src/account-settings/AccountSettingsPage.jsx index 328bde1..890d1af 100644 --- a/src/account-settings/AccountSettingsPage.jsx +++ b/src/account-settings/AccountSettingsPage.jsx @@ -192,6 +192,8 @@ class AccountSettingsPage extends React.Component { } if (values !== this.props.committedValues?.verified_name) { this.props.beginNameChange(formId); + } else { + this.props.saveSettings(formId, values); } } diff --git a/src/account-settings/certificate-preference/CertificatePreference.jsx b/src/account-settings/certificate-preference/CertificatePreference.jsx index 1c539b2..f044a26 100644 --- a/src/account-settings/certificate-preference/CertificatePreference.jsx +++ b/src/account-settings/certificate-preference/CertificatePreference.jsx @@ -76,7 +76,7 @@ function CertificatePreference({ } useEffect(() => { - if (fieldName === 'verifiedName') { + if (fieldName === 'verified_name') { setChecked(useVerifiedNameForCerts); } else { setChecked(!useVerifiedNameForCerts); diff --git a/src/account-settings/certificate-preference/test/CertificatePreference.test.jsx b/src/account-settings/certificate-preference/test/CertificatePreference.test.jsx index 1ff28ab..df96134 100644 --- a/src/account-settings/certificate-preference/test/CertificatePreference.test.jsx +++ b/src/account-settings/certificate-preference/test/CertificatePreference.test.jsx @@ -158,4 +158,17 @@ describe('NameChange', () => { type: 'ACCOUNT_SETTINGS__SAVE_SETTINGS', }); }); + + it('checks box for verified name', () => { + props = { + ...props, + fieldName: 'verified_name', + useVerifiedNameForCerts: true, + }; + + render(reduxWrapper()); + + const checkbox = screen.getByLabelText(labelText); + expect(checkbox.checked).toEqual(true); + }); }); diff --git a/src/account-settings/data/selectors.js b/src/account-settings/data/selectors.js index 7a1835c..94b7883 100644 --- a/src/account-settings/data/selectors.js +++ b/src/account-settings/data/selectors.js @@ -63,7 +63,7 @@ const valuesSelector = createSelector( mostRecentApprovedVerifiedNameValueSelector, (accountSettings, mostRecentApprovedVerifiedNameValue) => { let useVerifiedNameForCerts = ( - accountSettings.values.verifiedNameHistory?.use_verified_name_for_certs || false + accountSettings.verifiedNameHistory?.use_verified_name_for_certs || false ); if (Object.keys(accountSettings.confirmationValues).includes('useVerifiedNameForCerts')) {