From 0d399f8b1318c1f4bcc5dd6682b0800facdea187 Mon Sep 17 00:00:00 2001
From: Ole Martin Pettersen <olemp@puzzlepart.com>
Date: Fri, 17 Jan 2025 14:26:39 +0100
Subject: [PATCH]  113:7  error    Unexpected console statement             
 no-console

---
 .../src/webparts/portfolioOverview/index.ts   | 34 ++++++++++---------
 .../PortalDataService/PortalDataService.ts    |  4 +--
 2 files changed, 19 insertions(+), 19 deletions(-)

diff --git a/SharePointFramework/PortfolioWebParts/src/webparts/portfolioOverview/index.ts b/SharePointFramework/PortfolioWebParts/src/webparts/portfolioOverview/index.ts
index aae0d8cbd..00d5e0245 100644
--- a/SharePointFramework/PortfolioWebParts/src/webparts/portfolioOverview/index.ts
+++ b/SharePointFramework/PortfolioWebParts/src/webparts/portfolioOverview/index.ts
@@ -59,8 +59,8 @@ export default class PortfolioOverviewWebPart extends BasePortfolioWebPart<IPort
   public async onInit(): Promise<void> {
     try {
       this._selectedPortfolioId = this.properties.selectedPortfolioId
-      const selectedPortfolio = this.properties.portfolios.find((portfolio) => portfolio.uniqueId === this._selectedPortfolioId)
-      await super.onInit(selectedPortfolio)
+      const portfolio = this.properties.portfolios.find(({ uniqueId }) => uniqueId === this._selectedPortfolioId)
+      await super.onInit(portfolio)
       this._configuration = await this.dataAdapter.getPortfolioConfig()
     } catch (error) {
       this._error = error
@@ -69,13 +69,18 @@ export default class PortfolioOverviewWebPart extends BasePortfolioWebPart<IPort
 
   /**
    * Get dropdown options for the specified `targetProperty`. For now it only
-   * handles the `defaultViewId` property, but in the future it could be used
-   * to populate other dropdowns in the property pane.
+   * handles the `defaultViewId` and `portfolios` properties.
    *
    * @param targetProperty Target property
    */
-  protected _getOptions(targetProperty: string): IPropertyPaneDropdownOption[] {
+  protected _getOptions(targetProperty: keyof IPortfolioOverviewProps): IPropertyPaneDropdownOption[] {
     switch (targetProperty) {
+      case 'portfolios': {
+        return this.properties.portfolios.map((portfolio) => ({
+          key: portfolio.uniqueId,
+          text: portfolio.title
+        }))
+      }
       case 'defaultViewId':
         {
           if (this._configuration) {
@@ -104,17 +109,7 @@ export default class PortfolioOverviewWebPart extends BasePortfolioWebPart<IPort
                 !_.isEmpty(this.properties.portfolios) && (
                   PropertyPaneDropdown('selectedPortfolioId', {
                     label: strings.SelectedPortfolioLabel,
-                    options: this.properties.portfolios.map((portfolio) => ({
-                      key: portfolio.uniqueId,
-                      text: portfolio.title
-                    })),
-                  })
-                ),
-                !_.isEmpty(this.properties.portfolios) && (
-                  PropertyPaneToggle('showPortfolioSelector', {
-                    label: strings.ShowPortfolioSelectorLabel,
-                    onText: strings.ShowPortfolioSelectorOnText,
-                    offText: strings.ShowPortfolioSelectorOffText
+                    options: this._getOptions('portfolios')
                   })
                 ),
                 PropertyPaneDropdown('defaultViewId', {
@@ -139,6 +134,13 @@ export default class PortfolioOverviewWebPart extends BasePortfolioWebPart<IPort
                 PropertyPaneToggle('includeViewNameInExcelExportFilename', {
                   label: strings.IncludeViewNameInExcelExportFilenameLabel
                 }),
+                !_.isEmpty(this.properties.portfolios) && (
+                  PropertyPaneToggle('showPortfolioSelector', {
+                    label: strings.ShowPortfolioSelectorLabel,
+                    onText: strings.ShowPortfolioSelectorOnText,
+                    offText: strings.ShowPortfolioSelectorOffText
+                  })
+                ),
                 PropertyPaneToggle('showViewSelector', {
                   label: strings.ShowViewSelectorLabel
                 }),
diff --git a/SharePointFramework/shared-library/src/services/PortalDataService/PortalDataService.ts b/SharePointFramework/shared-library/src/services/PortalDataService/PortalDataService.ts
index 114a2779b..da205301b 100644
--- a/SharePointFramework/shared-library/src/services/PortalDataService/PortalDataService.ts
+++ b/SharePointFramework/shared-library/src/services/PortalDataService/PortalDataService.ts
@@ -108,9 +108,7 @@ export class PortalDataService extends DataService<IPortalDataServiceConfigurati
       return
     }
     try {
-      this.hubSiteId =
-        this._configuration.spfxContext.pageContext.legacyPageContext.hubSiteId || ''
-      console.log('onInit', this.hubSiteId)
+      this.hubSiteId = this._configuration.spfxContext.pageContext.legacyPageContext.hubSiteId || ''
       try {
         const hubSite = await (
           await fetch(