Merge pull request #1215 from topcoder-platform/pm-1503_4

fix(PM-1503): removed breadcrumb for non admin users

Author: hentrymartin

src/apps/review/src/pages/scorecards/ViewScorecardPage/ViewScorecardPage.tsx

@@ -20,8 +20,11 @@ const ViewScorecardPage: FC = () => {
     const { profile }: ProfileContextData = useContext(profileContext)
     const isAdmin = profile?.roles.includes(UserRole.administrator)
     const breadCrumb = useMemo(
-        () => [{ index: 1, label: 'Scorecards', path: '/scorecard' }, { index: 2, label: 'Scorecards Details' }],
-        [],
+        () => ((profile && isAdmin)
+            ? [{ index: 1, label: 'Scorecards', path: '/scorecard' }, { index: 2, label: 'Scorecards Details' }]
+            : []
+        ),
+        [profile, isAdmin],
     )
 
     const scorecardQuery = useFetchScorecard(scorecardId, true)