Merge pull request #4063 from topcoder-platform/develop

PR for private stats feature

Author: sushilshinde

.circleci/config.yml

@@ -174,15 +174,14 @@ 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
@@ -191,8 +190,7 @@ workflows:
           filters:
             branches:
               only:
-                - develop 
-                - feature-contentful
+                - develop
       # Production builds are exectuted only on tagged commits to the
       # master branch.
       - "build-prod":

CONTRIBUTING.md

@@ -68,4 +68,9 @@ 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.
+The rest of the commit message is then used for this.
+
+### Generate Change long
+```
+npm run release:changelog
+```

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

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

docs/deployment-env.md

@@ -48,3 +48,4 @@ 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.string.isRequired,
+  isVerified: PT.bool.isRequired,
 };
 
 export default Skill;

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

@@ -5,9 +5,8 @@
 import _ from 'lodash';
 import React from 'react';
 import PT from 'prop-types';
-import { Link } from 'react-router-dom';
 import ReactSVG from 'react-svg';
-import { isomorphy } from 'topcoder-react-utils';
+import { Link, 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';
@@ -50,7 +49,6 @@ class ProfileStats extends React.Component {
 
   render() {
     const {
-      stats,
       statsDistribution,
       statsHistory,
       track,
@@ -60,6 +58,11 @@ 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;
 
@@ -300,7 +303,7 @@ ProfileStats.defaultProps = {
 };
 
 ProfileStats.propTypes = {
-  stats: PT.shape().isRequired,
+  stats: PT.arrayOf(PT.shape()).isRequired,
   handleParam: PT.string.isRequired,
   track: PT.string.isRequired,
   subTrack: PT.string.isRequired,

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

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

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

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

src/shared/components/ProfilePage/index.jsx

@@ -63,7 +63,12 @@ class ProfilePage extends React.Component {
   }
 
   getActiveTracks() {
-    const { copilot, stats } = this.props;
+    const { copilot } = this.props;
+    let { stats } = this.props;
+    if (_.isArray(stats)) {
+      // eslint-disable-next-line prefer-destructuring
+      stats = stats[0];
+    }
     const activeTracks = [];
 
     if (copilot && stats && stats.COPILOT && stats.COPILOT.fulfillment) {
@@ -118,6 +123,7 @@ class ProfilePage extends React.Component {
       skills: propSkills,
       stats,
       lookupData,
+      meta,
     } = this.props;
 
     const {
@@ -257,7 +263,7 @@ class ProfilePage extends React.Component {
               {
                 stats && (
                   <div id="profile-activity">
-                    <StatsCategory handle={info.handle} stats={stats} />
+                    <StatsCategory handle={info.handle} stats={stats} meta={meta} />
                   </div>
                 )
               }
@@ -297,6 +303,7 @@ ProfilePage.defaultProps = {
   achievements: [],
   skills: null,
   stats: null,
+  meta: null,
 };
 
 ProfilePage.propTypes = {
@@ -306,8 +313,9 @@ ProfilePage.propTypes = {
   externalLinks: PT.arrayOf(PT.shape()),
   info: PT.shape().isRequired,
   skills: PT.shape(),
-  stats: PT.shape(),
+  stats: PT.arrayOf(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: `${BASE_URL}/members/${normalizedProfile.handle}`,
+        link: `${meta ? _.replace(BASE_URL, 'www', meta.subdomains[0]) : BASE_URL}/members/${normalizedProfile.handle}`,
         title: 'My Profile',
       }, {
         openNewTab: true,

src/shared/containers/Profile.jsx

@@ -1,6 +1,7 @@
 /**
  * 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';
@@ -15,20 +16,22 @@ class ProfileContainer extends React.Component {
     const {
       handleParam,
       loadProfile,
+      meta,
     } = this.props;
 
-    loadProfile(handleParam);
+    loadProfile(handleParam, _.join(_.get(meta, 'groupIds', [])));
   }
 
   componentWillReceiveProps(nextProps) {
     const {
       handleParam,
       profileForHandle,
       loadProfile,
+      meta,
     } = nextProps;
 
     if (handleParam !== profileForHandle) {
-      loadProfile(handleParam);
+      loadProfile(handleParam, _.join(_.get(meta, 'groupIds', [])));
     }
   }
 
@@ -75,6 +78,7 @@ ProfileContainer.defaultProps = {
   profileForHandle: '',
   skills: null,
   stats: null,
+  meta: null,
 };
 
 ProfileContainer.propTypes = {
@@ -89,8 +93,9 @@ ProfileContainer.propTypes = {
   loadProfile: PT.func.isRequired,
   profileForHandle: PT.string,
   skills: PT.shape(),
-  stats: PT.shape(),
+  stats: PT.arrayOf(PT.shape()),
   lookupData: PT.shape().isRequired,
+  meta: PT.shape(),
 };
 
 const mapStateToProps = (state, ownProps) => ({
@@ -100,6 +105,7 @@ 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,
@@ -112,7 +118,7 @@ function mapDispatchToProps(dispatch) {
   const a = actions.profile;
   const lookupActions = actions.lookup;
   return {
-    loadProfile: (handle) => {
+    loadProfile: (handle, groupIds) => {
       dispatch(a.clearProfile());
       dispatch(a.loadProfile(handle));
       dispatch(a.getAchievementsInit());
@@ -127,7 +133,7 @@ function mapDispatchToProps(dispatch) {
       dispatch(a.getExternalLinksDone(handle));
       dispatch(a.getInfoDone(handle));
       dispatch(a.getSkillsDone(handle));
-      dispatch(a.getStatsDone(handle));
+      dispatch(a.getStatsDone(handle, groupIds));
       dispatch(lookupActions.getCountriesDone());
     },
   };