Merge pull request #31976 from openedx/MJG/acc-edx-brand-removal

feat: Remove edX branding on account deletion
openedx · Feb 27, 2024 · d95ac60 · d95ac60
2 parents fca5141 + bdb3c9f
commit d95ac60
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 20 deletions.
diff --git a/lms/static/js/student_account/components/StudentAccountDeletion.jsx b/lms/static/js/student_account/components/StudentAccountDeletion.jsx
@@ -38,13 +38,7 @@ export class StudentAccountDeletion extends React.Component {
 
     render() {
         const { deletionModalOpen, socialAuthConnected, isActive } = this.state;
-        const loseAccessText = StringUtils.interpolate(
-            gettext('You may also lose access to verified certificates and other program credentials like MicroMasters certificates. If you want to make a copy of these for your records before proceeding with deletion, follow the instructions for {htmlStart}printing or downloading a certificate{htmlEnd}.'),
-            {
-                htmlStart: '<a href="https://edx.readthedocs.io/projects/open-edx-learner-guide/en/latest/OpenSFD_certificates.html#print-a-web-certificate" rel="noopener" target="_blank">',
-                htmlEnd: '</a>',
-            },
-        );
+        const loseAccessText = gettext('You may also lose access to verified certificates and other program credentials. You can make a copy of these for your records before proceeding with deletion.')
 
         const showError = socialAuthConnected || !isActive;
 
@@ -113,11 +107,7 @@ export class StudentAccountDeletion extends React.Component {
                     <span>{bodyDeletion} </span>
                     <span>{bodyDeletion2}</span>
                 </p>
-                <p
-                    className="account-settings-header-subtitle"
-                    // eslint-disable-next-line react/no-danger
-                    dangerouslySetInnerHTML={{ __html: loseAccessText }}
-                />
+                <p className="account-settings-header-subtitle">{loseAccessText}</p>
                 <p
                     className="account-settings-header-subtitle-warning"
                     // eslint-disable-next-line react/no-danger

diff --git a/lms/static/js/student_account/components/StudentAccountDeletionModal.jsx b/lms/static/js/student_account/components/StudentAccountDeletionModal.jsx
@@ -94,13 +94,7 @@ class StudentAccountDeletionConfirmationModal extends React.Component {
             validationMessage,
         } = this.state;
         const { onClose } = this.props;
-        const loseAccessText = StringUtils.interpolate(
-            gettext('You may also lose access to verified certificates and other program credentials like MicroMasters certificates. If you want to make a copy of these for your records before proceeding with deletion, follow the instructions for {htmlStart}printing or downloading a certificate{htmlEnd}.'),
-            {
-                htmlStart: '<a href="https://edx.readthedocs.io/projects/open-edx-learner-guide/en/latest/OpenSFD_certificates.html#print-a-web-certificate" rel="noopener" target="_blank">',
-                htmlEnd: '</a>',
-            },
-        );
+        const loseAccessText = gettext('You may also lose access to verified certificates and other program credentials. You can make a copy of these for your records before proceeding with deletion.')
 
         const noteDeletion = StringUtils.interpolate(
             gettext('You have selected “Delete my account.” Deletion of your account and personal data is permanent and cannot be undone. {platformName} will not be able to recover your account or the data that is deleted.'),
@@ -168,7 +162,7 @@ class StudentAccountDeletionConfirmationModal extends React.Component {
                                                 <span>{bodyDeletion2}</span>
                                             </p>
                                             {/* eslint-disable-next-line react/no-danger */}
-                                            <p dangerouslySetInnerHTML={{ __html: loseAccessText }} />
+                                            <p>{loseAccessText}</p>
                                         </div>
                                     </div>
                                 )}