Merge pull request #1654 from appirio-tech/bugBash07

Bug bash07- partial merge

Author: gondzo

src/api/projectMembers.js

@@ -5,7 +5,7 @@ import { TC_API_URL, PROJECTS_API_URL, CONNECT_MESSAGE_API_URL } from '../config
 export function getMembersById (userIds) {
   const _userIdArr = _.map(userIds, _id => `userId:${_id}`)
   // only requesting certain member attributes
-  const fields = 'userId,handle,photoURL,firstName,lastName,maxRating,details,email'
+  const fields = 'userId,handle,photoURL,firstName,lastName,details,email'
   const query = _userIdArr.join(' OR ')
   const url = `${TC_API_URL}/v3/members/_search/?fields=`
     + encodeURIComponent(fields)

src/components/TopBar/ProjectsToolBar.js

@@ -145,7 +145,7 @@ class ProjectsToolBar extends Component {
       excludedFiltersCount++
     }
     // Ignore status from filters count
-    const noOfFilters = _.keys(_.omit(criteria, ['status'])).length - excludedFiltersCount
+    const noOfFilters = _.keys(_.omit(criteria, ['status', 'keyword'])).length - excludedFiltersCount
     const onLeaveMessage = this.onLeave() || ''
 
     const primaryNavigationItems = [

src/components/User/UserAvatar.jsx

@@ -1,15 +1,18 @@
 import React from 'react'
 import PropTypes from 'prop-types'
 import { DOMAIN } from '../../config/constants'
-import UserAvatar from './UserAvatar'
+import { Avatar } from 'appirio-tech-react-components'
 require('./UserWithName.scss')
 
-const UserWithName = ({ handle, firstName, lastName, photoURL, maxRating={rating: 0}, showLevel=false }) => {
+const UserWithName = ({ handle, firstName, lastName, photoURL }) => {
   const url = `//www.${DOMAIN}/members/${handle}/`
   return (
     <div className="user-block">
       <a href={url} target="_blank" rel="noopener noreferrer" className="photo">
-        <UserAvatar rating={maxRating.rating} showLevel={showLevel} photoURL={photoURL} />
+        <Avatar
+          avatarUrl={photoURL}
+          userName={firstName + ' ' + lastName}
+        />
       </a>
       <span className="txt-box">
         <a href={url} target="_blank" rel="noopener noreferrer" className="link-black">{firstName} {lastName}</a>
@@ -23,9 +26,7 @@ UserWithName.propTypes = {
   handle  : PropTypes.string.isRequired,
   firstName : PropTypes.string.isRequired,
   lastName  : PropTypes.string.isRequired,
-  photoURL  : PropTypes.string,
-  maxRating : PropTypes.object,
-  showLevel : PropTypes.bool
+  photoURL  : PropTypes.string
 }
 
 export default UserWithName

src/components/User/UserWithName.jsx

@@ -61,7 +61,7 @@ class ProjectListProjectColHeader extends React.Component {
                   const activeClass = cn({
                     active: item.val === currentSortField
                   })
-                  return (<li key={i} className={activeClass} onClick={sortHandler}>
+                  return (<li key={i} className={activeClass} onClick={() => sortHandler(item.val)}>
                     {activeClass? <IconCheckDark className="icon-check-dark"/>: ''}
                     <a href="javascript:;">{item.label}</a>
                   </li>)