Merge pull request #4064 from topcoder-platform/revert-4063-develop

Revert "PR for private stats feature"

Author: sushilshinde

.circleci/config.yml

@@ -174,14 +174,15 @@ workflows:
           filters:
             branches:
               only:
-                - feature-m2m-with-stats
                 - develop
+                - feature-m2m-token
       # This is alternate dev env for parallel testing
       - "build-test":
           context : org-global      
           filters:
             branches:
               only:
+                - develop
                 - nav-hot-fix
                 - hot-fixes-leaderboard
       # This is beta env for production soft releases
@@ -190,7 +191,8 @@ workflows:
           filters:
             branches:
               only:
-                - develop
+                - develop 
+                - feature-contentful
       # Production builds are exectuted only on tagged commits to the
       # master branch.
       - "build-prod":

CONTRIBUTING.md

@@ -68,9 +68,4 @@ Optional. The footer should contain any information about **Breaking Changes** a
 reference GitHub issues that this commit **Closes**.
 
 **Breaking Changes** should start with the word `BREAKING CHANGE:` with a space or two newlines.
-The rest of the commit message is then used for this.
-
-### Generate Change long
-```
-npm run release:changelog
-```
+The rest of the commit message is then used for this.

__tests__/shared/components/ProfilePage/__snapshots__/index.jsx.snap

@@ -199,7 +199,6 @@ exports[`renders a full Profile correctly 1`] = `
             className=""
             handle="testHandle"
             inModal={false}
-            meta={null}
             stats={
               Object {
                 "COPILOT": Object {
@@ -700,7 +699,6 @@ exports[`renders an empty Profile correctly 1`] = `
             className=""
             handle="testHandle"
             inModal={false}
-            meta={null}
             stats={
               Object {
                 "COPILOT": null,

docs/deployment-env.md

@@ -48,4 +48,3 @@ workflows:
                 - develop
 ```
 4. Commit the changes
-5. Status of the deployment environments can be checked here https://cci-reporter.herokuapp.com/

src/shared/components/ProfilePage/Skill/index.jsx

@@ -37,7 +37,7 @@ const Skill = ({
 Skill.propTypes = {
   tagId: PT.string.isRequired,
   tagName: PT.string.isRequired,
-  isVerified: PT.bool.isRequired,
+  isVerified: PT.string.isRequired,
 };
 
 export default Skill;

src/shared/components/ProfilePage/Stats/index.jsx

@@ -5,8 +5,9 @@
 import _ from 'lodash';
 import React from 'react';
 import PT from 'prop-types';
+import { Link } from 'react-router-dom';
 import ReactSVG from 'react-svg';
-import { Link, isomorphy } from 'topcoder-react-utils';
+import { isomorphy } from 'topcoder-react-utils';
 import { getRatingColor } from 'utils/tc';
 import Th from 'assets/images/th.svg';
 import LeftArrow from 'assets/images/arrow-prev.svg';
@@ -49,6 +50,7 @@ class ProfileStats extends React.Component {
 
   render() {
     const {
+      stats,
       statsDistribution,
       statsHistory,
       track,
@@ -58,11 +60,6 @@ class ProfileStats extends React.Component {
       handleParam,
       activeChallengesCount,
     } = this.props;
-    let { stats } = this.props;
-    if (_.isArray(stats)) {
-      // eslint-disable-next-line prefer-destructuring
-      stats = stats[0];
-    }
 
     const { activeGraph, showModal } = this.state;
 
@@ -303,7 +300,7 @@ ProfileStats.defaultProps = {
 };
 
 ProfileStats.propTypes = {
-  stats: PT.arrayOf(PT.shape()).isRequired,
+  stats: PT.shape().isRequired,
   handleParam: PT.string.isRequired,
   track: PT.string.isRequired,
   subTrack: PT.string.isRequired,

src/shared/components/ProfilePage/StatsCategory/index.jsx

@@ -43,11 +43,7 @@ const isActiveSubtrack = (subtrack) => {
 
 class StatsCategory extends React.Component {
   getActiveTracks() {
-    let { stats } = this.props;
-    if (_.isArray(stats)) {
-      // eslint-disable-next-line prefer-destructuring
-      stats = stats[0];
-    }
+    const { stats } = this.props;
     const activeTracks = [];
 
     if (stats.COPILOT && stats.COPILOT.fulfillment) {
@@ -93,7 +89,6 @@ class StatsCategory extends React.Component {
       handle,
       className,
       inModal,
-      meta,
     } = this.props;
 
     const activeTracks = this.getActiveTracks();
@@ -118,7 +113,7 @@ class StatsCategory extends React.Component {
                   <Link
                     to={`/members/${handle}/details/?track=${track.name}&subTrack=${subtrack.name.replace(' ', '_')}`}
                     key={subtrack.name}
-                    styleName={`subtrack ${index === 0 ? 'first' : ''} ${meta ? 'disablelink' : ''}`}
+                    styleName={`subtrack ${index === 0 ? 'first' : ''}`}
                   >
                     <div
                       styleName="name"
@@ -184,15 +179,13 @@ class StatsCategory extends React.Component {
 StatsCategory.defaultProps = {
   className: '',
   inModal: false,
-  meta: null,
 };
 
 StatsCategory.propTypes = {
   handle: PT.string.isRequired,
-  stats: PT.arrayOf(PT.shape()).isRequired,
+  stats: PT.shape().isRequired,
   inModal: PT.bool,
   className: PT.string,
-  meta: PT.shape(),
 };
 
 export default StatsCategory;

src/shared/components/ProfilePage/StatsCategory/styles.scss

@@ -147,10 +147,6 @@
       }
     }
 
-    .disablelink {
-      pointer-events: none;
-    }
-
     .icon {
       width: 80px;
       height: 80px;

src/shared/components/ProfilePage/index.jsx

@@ -63,12 +63,7 @@ class ProfilePage extends React.Component {
   }
 
   getActiveTracks() {
-    const { copilot } = this.props;
-    let { stats } = this.props;
-    if (_.isArray(stats)) {
-      // eslint-disable-next-line prefer-destructuring
-      stats = stats[0];
-    }
+    const { copilot, stats } = this.props;
     const activeTracks = [];
 
     if (copilot && stats && stats.COPILOT && stats.COPILOT.fulfillment) {
@@ -123,7 +118,6 @@ class ProfilePage extends React.Component {
       skills: propSkills,
       stats,
       lookupData,
-      meta,
     } = this.props;
 
     const {
@@ -263,7 +257,7 @@ class ProfilePage extends React.Component {
               {
                 stats && (
                   <div id="profile-activity">
-                    <StatsCategory handle={info.handle} stats={stats} meta={meta} />
+                    <StatsCategory handle={info.handle} stats={stats} />
                   </div>
                 )
               }
@@ -303,7 +297,6 @@ ProfilePage.defaultProps = {
   achievements: [],
   skills: null,
   stats: null,
-  meta: null,
 };
 
 ProfilePage.propTypes = {
@@ -313,9 +306,8 @@ ProfilePage.propTypes = {
   externalLinks: PT.arrayOf(PT.shape()),
   info: PT.shape().isRequired,
   skills: PT.shape(),
-  stats: PT.arrayOf(PT.shape()),
+  stats: PT.shape(),
   lookupData: PT.shape().isRequired,
-  meta: PT.shape(),
 };
 
 export default ProfilePage;

src/shared/components/tc-communities/Header/index.jsx

@@ -66,7 +66,7 @@ function Header(props) {
       items: [{
         enforceA: true,
         icon: <IconNavProfile />,
-        link: `${meta ? _.replace(BASE_URL, 'www', meta.subdomains[0]) : BASE_URL}/members/${normalizedProfile.handle}`,
+        link: `${BASE_URL}/members/${normalizedProfile.handle}`,
         title: 'My Profile',
       }, {
         openNewTab: true,

src/shared/containers/Profile.jsx

@@ -1,7 +1,6 @@
 /**
  * Connects the Redux store to the Profile display components.
  */
-import _ from 'lodash';
 import React from 'react';
 import PT from 'prop-types';
 import { connect } from 'react-redux';
@@ -16,22 +15,20 @@ class ProfileContainer extends React.Component {
     const {
       handleParam,
       loadProfile,
-      meta,
     } = this.props;
 
-    loadProfile(handleParam, _.join(_.get(meta, 'groupIds', [])));
+    loadProfile(handleParam);
   }
 
   componentWillReceiveProps(nextProps) {
     const {
       handleParam,
       profileForHandle,
       loadProfile,
-      meta,
     } = nextProps;
 
     if (handleParam !== profileForHandle) {
-      loadProfile(handleParam, _.join(_.get(meta, 'groupIds', [])));
+      loadProfile(handleParam);
     }
   }
 
@@ -78,7 +75,6 @@ ProfileContainer.defaultProps = {
   profileForHandle: '',
   skills: null,
   stats: null,
-  meta: null,
 };
 
 ProfileContainer.propTypes = {
@@ -93,9 +89,8 @@ ProfileContainer.propTypes = {
   loadProfile: PT.func.isRequired,
   profileForHandle: PT.string,
   skills: PT.shape(),
-  stats: PT.arrayOf(PT.shape()),
+  stats: PT.shape(),
   lookupData: PT.shape().isRequired,
-  meta: PT.shape(),
 };
 
 const mapStateToProps = (state, ownProps) => ({
@@ -105,7 +100,6 @@ const mapStateToProps = (state, ownProps) => ({
   externalAccounts: state.profile.externalAccounts,
   externalLinks: state.profile.externalLinks,
   handleParam: ownProps.match.params.handle,
-  meta: ownProps.meta,
   info: state.profile.info,
   loadingError: state.profile.loadingError,
   profileForHandle: state.profile.profileForHandle,
@@ -118,7 +112,7 @@ function mapDispatchToProps(dispatch) {
   const a = actions.profile;
   const lookupActions = actions.lookup;
   return {
-    loadProfile: (handle, groupIds) => {
+    loadProfile: (handle) => {
       dispatch(a.clearProfile());
       dispatch(a.loadProfile(handle));
       dispatch(a.getAchievementsInit());
@@ -133,7 +127,7 @@ function mapDispatchToProps(dispatch) {
       dispatch(a.getExternalLinksDone(handle));
       dispatch(a.getInfoDone(handle));
       dispatch(a.getSkillsDone(handle));
-      dispatch(a.getStatsDone(handle, groupIds));
+      dispatch(a.getStatsDone(handle));
       dispatch(lookupActions.getCountriesDone());
     },
   };