From 7c4676d0a45281bd4e5517454dcb532df67d6e10 Mon Sep 17 00:00:00 2001
From: Mrugesh Mohapatra <1884376+raisedadead@users.noreply.github.com>
Date: Sat, 8 May 2021 01:09:42 +0530
Subject: [PATCH] fix(client): quick-fix for cert claiming (#42055)

* chore: reinstate the quick-fix for cert claiming

This reverts commit d80d093c8bd2ee9f24dace9e98c50eae9cdb7cfa.

* feat: add progress to claim cert card
---
 client/i18n/locales/english/intro.json        |  1 +
 client/i18n/locales/english/translations.json | 10 ++
 .../src/components/settings/Certification.js  |  9 +-
 client/src/components/settings/Honesty.js     |  2 +-
 client/src/components/settings/Privacy.js     |  2 +-
 .../__snapshots__/Honesty.test.js.snap        |  2 +
 .../templates/Introduction/SuperBlockIntro.js |  1 +
 .../Introduction/components/CertChallenge.js  | 11 +++
 .../components/CertificationCard.js           | 67 +++++++++++++
 .../Introduction/components/ClaimCertSteps.js | 99 +++++++++++++++++++
 10 files changed, 200 insertions(+), 4 deletions(-)
 create mode 100644 client/src/templates/Introduction/components/CertificationCard.js
 create mode 100644 client/src/templates/Introduction/components/ClaimCertSteps.js

diff --git a/client/i18n/locales/english/intro.json b/client/i18n/locales/english/intro.json
index d39ff3d0e4..46227ae015 100644
--- a/client/i18n/locales/english/intro.json
+++ b/client/i18n/locales/english/intro.json
@@ -506,6 +506,7 @@
     "certification": "{{cert}} Certification",
     "browse-other": "Browse our other free certifications\n(we recommend doing these in order)",
     "courses": "Courses",
+    "steps": "Steps",
     "expand": "Expand",
     "collapse": "Collapse"
   }
diff --git a/client/i18n/locales/english/translations.json b/client/i18n/locales/english/translations.json
index 15bd0852f0..99e0a7589a 100644
--- a/client/i18n/locales/english/translations.json
+++ b/client/i18n/locales/english/translations.json
@@ -554,5 +554,15 @@
         "Neural Network SMS Text Classifier": "Neural Network SMS Text Classifier"
       }
     }
+  },
+  "certification-card": {
+    "title": "Claim Your Certification",
+    "intro": "Complete the following steps to claim and view your {{i18nCertText}}",
+    "complete-project": "Complete {{i18nCertText}} Projects",
+    "accept-honesty": "Accept our Academic Honesty Policy",
+    "set-name": "Set your name, and make it public",
+    "set-certs-public": "Set your certificication settings to public",
+    "set-profile-public": "Set your profile settings to public",
+    "set-claim": "Claim and view your certification"
   }
 }
diff --git a/client/src/components/settings/Certification.js b/client/src/components/settings/Certification.js
index bfe08b93b6..7f8d67eb21 100644
--- a/client/src/components/settings/Certification.js
+++ b/client/src/components/settings/Certification.js
@@ -260,10 +260,13 @@ export class CertificationSettings extends Component {
 
   renderCertifications = certName => {
     const { t } = this.props;
+    const { certSlug } = first(projectMap[certName]);
     return (
       <FullWidthRow key={certName}>
         <Spacer />
-        <h3 className='text-center'>{certName}</h3>
+        <h3 className='text-center' id={`cert-${certSlug}`}>
+          {certName}
+        </h3>
         <Table>
           <thead>
             <tr>
@@ -448,7 +451,9 @@ export class CertificationSettings extends Component {
     return (
       <FullWidthRow key={certSlug}>
         <Spacer />
-        <h3 className='text-center'>{certName}</h3>
+        <h3 className='text-center' id={`cert-${certSlug}`}>
+          {certName}
+        </h3>
         <Form
           buttonText={
             fullForm ? t('buttons.claim-cert') : t('buttons.save-progress')
diff --git a/client/src/components/settings/Honesty.js b/client/src/components/settings/Honesty.js
index 2ff6948629..90ddd2c33f 100644
--- a/client/src/components/settings/Honesty.js
+++ b/client/src/components/settings/Honesty.js
@@ -35,7 +35,7 @@ const Honesty = ({ isHonest, updateIsHonest }) => {
     </Button>
   );
   return (
-    <section className='honesty-policy'>
+    <section className='honesty-policy' id='honesty-policy'>
       <SectionHeader>{t('settings.headings.honesty')}</SectionHeader>
       <FullWidthRow>
         <Panel className='honesty-panel'>
diff --git a/client/src/components/settings/Privacy.js b/client/src/components/settings/Privacy.js
index b0f1d745c2..0e34243975 100644
--- a/client/src/components/settings/Privacy.js
+++ b/client/src/components/settings/Privacy.js
@@ -66,7 +66,7 @@ class PrivacySettings extends Component {
     } = user.profileUI;
 
     return (
-      <div className='privacy-settings'>
+      <div className='privacy-settings' id='privacy-settings'>
         <SectionHeader>{t('settings.headings.privacy')}</SectionHeader>
         <FullWidthRow>
           <p>{t('settings.privacy')}</p>
diff --git a/client/src/components/settings/__snapshots__/Honesty.test.js.snap b/client/src/components/settings/__snapshots__/Honesty.test.js.snap
index c88fda569d..9e6d5accbd 100644
--- a/client/src/components/settings/__snapshots__/Honesty.test.js.snap
+++ b/client/src/components/settings/__snapshots__/Honesty.test.js.snap
@@ -3,6 +3,7 @@
 exports[`<Honesty /> <Honesty /> snapshot when isHonest is false: Honesty 1`] = `
 <section
   className="honesty-policy"
+  id="honesty-policy"
 >
   <SectionHeader>
     settings.headings.honesty
@@ -31,6 +32,7 @@ exports[`<Honesty /> <Honesty /> snapshot when isHonest is false: Honesty 1`] =
 exports[`<Honesty /> <Honesty /> snapshot when isHonest is true: HonestyAccepted 1`] = `
 <section
   className="honesty-policy"
+  id="honesty-policy"
 >
   <SectionHeader>
     settings.headings.honesty
diff --git a/client/src/templates/Introduction/SuperBlockIntro.js b/client/src/templates/Introduction/SuperBlockIntro.js
index 4d0d104c01..147b0373ee 100644
--- a/client/src/templates/Introduction/SuperBlockIntro.js
+++ b/client/src/templates/Introduction/SuperBlockIntro.js
@@ -180,6 +180,7 @@ class SuperBlockIntroductionPage extends Component {
                 {superBlock !== 'coding-interview-prep' && (
                   <div>
                     <CertChallenge
+                      isSignedIn={isSignedIn}
                       superBlock={superBlock}
                       title={title}
                       user={user}
diff --git a/client/src/templates/Introduction/components/CertChallenge.js b/client/src/templates/Introduction/components/CertChallenge.js
index 06399bffd0..d5c7a8b57e 100644
--- a/client/src/templates/Introduction/components/CertChallenge.js
+++ b/client/src/templates/Introduction/components/CertChallenge.js
@@ -15,9 +15,11 @@ import {
   certSlugTypeMap,
   superBlockCertTypeMap
 } from '../../../../../config/certification-settings';
+import CertificationCard from './CertificationCard';
 
 const propTypes = {
   currentCerts: CurrentCertsType,
+  isSignedIn: PropTypes.bool.isRequired,
   superBlock: PropTypes.string,
   t: PropTypes.func,
   title: PropTypes.string,
@@ -36,6 +38,7 @@ const mapStateToProps = (state, props) => {
 export class CertChallenge extends Component {
   render() {
     const {
+      isSignedIn,
       superBlock,
       t,
       title,
@@ -54,9 +57,17 @@ export class CertChallenge extends Component {
     const i18nCertText = t(`intro:misc-text.certification`, {
       cert: i18nSuperBlock
     });
+    const { certSlug } = cert;
 
     return (
       <div className='block'>
+        {isSignedIn && !isCertified && (
+          <CertificationCard
+            certSlug={certSlug}
+            i18nCertText={i18nCertText}
+            superBlock={superBlock}
+          />
+        )}
         <button
           className={`map-cert-title ${
             isCertified ? 'map-is-cert' : 'no-cursor'
diff --git a/client/src/templates/Introduction/components/CertificationCard.js b/client/src/templates/Introduction/components/CertificationCard.js
new file mode 100644
index 0000000000..725db4a3e5
--- /dev/null
+++ b/client/src/templates/Introduction/components/CertificationCard.js
@@ -0,0 +1,67 @@
+import React, { useState } from 'react';
+import PropTypes from 'prop-types';
+import ScrollableAnchor from 'react-scrollable-anchor';
+import { useTranslation } from 'react-i18next';
+
+import ClaimCertSteps from './ClaimCertSteps';
+import Caret from '../../../assets/icons/Caret';
+
+const propTypes = {
+  certSlug: PropTypes.string,
+  i18nCertText: PropTypes.string,
+  superBlock: PropTypes.string
+};
+
+const CertificationCard = ({ certSlug, superBlock, i18nCertText }) => {
+  const { t } = useTranslation();
+  const [isExpanded, setIsExpanded] = useState(true);
+
+  const handleBlockClick = () => {
+    setIsExpanded(!isExpanded);
+  };
+
+  const { expand: expandText, collapse: collapseText, steps: stepsText } = t(
+    'intro:misc-text'
+  );
+  return (
+    <ScrollableAnchor id='claim-cert-block'>
+      <div className={`block ${isExpanded ? 'open' : ''}`}>
+        <div className='block-title-wrapper'>
+          <a className='block-link' href='#claim-cert-block'>
+            <h3 className='big-block-title'>
+              {t('certification-card.title')}
+              <span className='block-link-icon'>#</span>
+            </h3>
+          </a>
+        </div>
+        <div className='block-description'>
+          {t('certification-card.intro', { i18nCertText })}
+        </div>
+        <button
+          aria-expanded={isExpanded}
+          className='map-title'
+          onClick={handleBlockClick}
+        >
+          <Caret />
+          <h4 className='course-title'>
+            {`${
+              isExpanded ? collapseText : expandText
+            } ${stepsText.toLowerCase()}`}
+          </h4>
+        </button>
+        {isExpanded && (
+          <ClaimCertSteps
+            certSlug={certSlug}
+            i18nCertText={i18nCertText}
+            superBlock={superBlock}
+          />
+        )}
+      </div>
+    </ScrollableAnchor>
+  );
+};
+
+CertificationCard.displayName = 'CertStatus';
+CertificationCard.propTypes = propTypes;
+
+export default CertificationCard;
diff --git a/client/src/templates/Introduction/components/ClaimCertSteps.js b/client/src/templates/Introduction/components/ClaimCertSteps.js
new file mode 100644
index 0000000000..c7f102b0d8
--- /dev/null
+++ b/client/src/templates/Introduction/components/ClaimCertSteps.js
@@ -0,0 +1,99 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+import { Link } from 'gatsby';
+import { withTranslation, useTranslation } from 'react-i18next';
+import { connect } from 'react-redux';
+import { createSelector } from 'reselect';
+
+import GreenPass from '../../../assets/icons/GreenPass';
+import GreenNotCompleted from '../../../assets/icons/GreenNotCompleted';
+import { userSelector } from '../../../redux';
+import { User } from '../../../redux/propTypes';
+
+const mapIconStyle = { height: '15px', marginRight: '10px', width: '15px' };
+const renderCheckMark = isCompleted => {
+  return isCompleted ? (
+    <GreenPass style={mapIconStyle} />
+  ) : (
+    <GreenNotCompleted style={mapIconStyle} />
+  );
+};
+
+const propTypes = {
+  certSlug: PropTypes.string,
+  i18nCertText: PropTypes.string,
+  superBlock: PropTypes.string,
+  user: User
+};
+
+const mapStateToProps = state => {
+  return createSelector(userSelector, user => ({
+    user
+  }))(state);
+};
+
+const ClaimCertSteps = ({ certSlug, i18nCertText, superBlock, user }) => {
+  const { t } = useTranslation();
+
+  const settingsLink = '/settings#privacy-settings';
+  const certClaimLink = `/settings#cert-${certSlug}`;
+  const honestyPolicyAnchor = '/settings#honesty-policy';
+
+  const {
+    name,
+    isHonest,
+    profileUI: { isLocked, showCerts, showName }
+  } = user;
+
+  return (
+    <ul className='map-challenges-ul'>
+      <li className='map-challenge-title map-project-wrap'>
+        <Link to={honestyPolicyAnchor}>
+          {t('certification-card.accept-honesty')}
+          <span className='badge map-badge map-project-checkmark'>
+            {renderCheckMark(isHonest)}
+          </span>
+        </Link>
+      </li>
+      <li className='map-challenge-title map-project-wrap'>
+        <Link to={settingsLink}>
+          {t('certification-card.set-profile-public')}
+          <span className='badge map-badge map-project-checkmark'>
+            {renderCheckMark(!isLocked)}
+          </span>
+        </Link>
+      </li>
+      <li className='map-challenge-title map-project-wrap'>
+        <Link to={settingsLink}>
+          {t('certification-card.set-certs-public')}
+          <span className='badge map-badge map-project-checkmark'>
+            {renderCheckMark(showCerts)}
+          </span>
+        </Link>
+      </li>
+      <li className='map-challenge-title map-project-wrap'>
+        <Link to={settingsLink}>
+          {t('certification-card.set-name')}
+          <span className='badge map-badge map-project-checkmark'>
+            {renderCheckMark(name && name !== '' && showName)}
+          </span>
+        </Link>
+      </li>
+      <li className='map-challenge-title map-project-wrap'>
+        <a href={`#${superBlock}-projects`}>
+          {t('certification-card.complete-project', {
+            i18nCertText
+          })}
+        </a>
+      </li>
+      <li className='map-challenge-title map-project-wrap'>
+        <Link to={certClaimLink}>{t('certification-card.set-claim')}</Link>
+      </li>
+    </ul>
+  );
+};
+
+ClaimCertSteps.displayName = 'ClaimCertSteps';
+ClaimCertSteps.propTypes = propTypes;
+
+export default connect(mapStateToProps)(withTranslation()(ClaimCertSteps));