Various improvements to check if user isViewAuthor

Puzzlepart · Nov 5, 2024 · 83f7896 · 83f7896
1 parent 6e59f48
commit 83f7896
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/...tfolioWebParts/src/components/PortfolioOverview/ColumnContextMenu/useColumnContextMenu.ts b/...tfolioWebParts/src/components/PortfolioOverview/ColumnContextMenu/useColumnContextMenu.ts
@@ -20,7 +20,8 @@ import { MenuProps, useId } from '@fluentui/react-components'
 export function useColumnContextMenu() {
   const context = useContext(PortfolioOverviewContext)
   const [open, setOpen] = useState(false)
-  const isViewAuthor = context.state.currentView?.author === context.props.pageContext.user.email
+  const userEmail = context.props.pageContext.user.email ?? context.props.pageContext.user.loginName
+  const isViewAuthor = context.state.currentView?.author === userEmail
 
   const { isAddColumn, createContextualMenuItems } = useAddColumn(
     true,

diff --git a/...mework/PortfolioWebParts/src/components/PortfolioOverview/ToolbarItems/useToolbarItems.ts b/...mework/PortfolioWebParts/src/components/PortfolioOverview/ToolbarItems/useToolbarItems.ts
@@ -45,8 +45,8 @@ const Icons = {
 export function useToolbarItems(context: IPortfolioOverviewContext) {
   const userCanManageViews = context.props.configuration.userCanAddViews
   const userCanEditGlobalViews = userCanManageViews && context.props.isSiteAdmin
-  const isViewAuthor =
-    userCanManageViews && context.state.currentView?.author === context.props.pageContext.user.email
+  const userEmail = context.props.pageContext.user.email ?? context.props.pageContext.user.loginName
+  const isViewAuthor = userCanManageViews && context.state.currentView?.author === userEmail
 
   const { exportToExcel } = useExcelExport(context)
   const sharedViews = useViewsMenuItems(context, (view) => !view.isPersonal)