Skip to content

Commit 2b7ec7c

Browse files
priyanshu92amitjoshi438amitjoshiashwani123psparrow1303
authored
Merge commits from main (#1165)
* Refactor PowerPagesChatParticipant and add CommandRegistry (#1050) Co-authored-by: amitjoshi <[email protected]> * [PowerPage][Copilot] Nl2Site and Nl2Page Service Integration for Site Create (#1053) * Refactor PowerPagesChatParticipantConstants and add NL2PAGE and NL2SITE constants * Refactor PowerPagesChatParticipantConstants and add new page types * Update constants and move telemetry to different file * Refactor and add getCommonHeaders function --------- Co-authored-by: amitjoshi <[email protected]> * Corrected scope of PPAPI access token for preprod env (#1055) Co-authored-by: Ashwani Kumar <[email protected]> * Hook for create site command with nl2page&site (#1057) * Hook for create site command with nl2page&site * Disable any type validation * Refactor PowerPagesChatParticipantConstants and CreateSiteCommand - Add NL2SITE_GENERATING_SITE constant for generating a new Power Pages site - Update progress message in CreateSiteCommand to use NL2SITE_GENERATING_SITE constant - Remove unnecessary markdown formatting in CreateSiteHelper error handling * Refactor localization files and update Power Pages chat participant utils * Refactor NL2SiteService and Nl2PageService to include additional telemetry logging --------- Co-authored-by: amitjoshi <[email protected]> Co-authored-by: tyaginidhi <[email protected]> * Add environment list retrieval and refactor BAP endpoint logic (#1060) * Add environment list retrieval and refactor BAP endpoint logic * Add type annotation for environment list retrieval in Utils.ts * Update src/common/utilities/Utils.ts Co-authored-by: Priyanshu Agrawal <[email protected]> * Refactor BAP environment list URL to use a variable for API version and update imports in Utils.ts --------- Co-authored-by: amitjoshi <[email protected]> Co-authored-by: Priyanshu Agrawal <[email protected]> * [PowerPages][create-site] Preview and Edit Site Page and Command Registration (#1061) * Enhance CreateSiteCommand to include extension context and add ReadonlyFileSystemProvider for site page previews * Implement EditableFileSystemProvider for site page editing and update CreateSiteHelper to utilize it * Integrate CreateSiteCommand into CommandRegistry and update related components for site creation functionality * Disable copy functionality in EditableFileSystemProvider implementation * Remove ReadonlyFileSystemProvider implementation * Add telemetry constant for previewing site pages and refactor related components * Refactor CommandRegistry and add command registration utility for chat participants * Add constants for site creation parameters and refactor NL2SiteService to use them * Refactor CreateSiteCommand and CreateSiteHelper to use structured options and improve readability; add CreateSiteTypes for better type management * Add error telemetry constant for previewing site pages and handle errors in previewSitePagesContent function * Rename fileContentMap to _fileContentMap for consistency and clarity in EditableFileSystemProvider * Remove unused getUpdatedPageContent function from CreateSiteHelper to streamline code * Add ESLint disable comments for any type usage in CreateSiteHelper and CreateSiteTypes --------- Co-authored-by: amitjoshi <[email protected]> * Bump cross-spawn (#1059) Bumps and [cross-spawn](https://github.com/moxystudio/node-cross-spawn). These dependencies needed to be updated together. Updates `cross-spawn` from 7.0.3 to 7.0.6 - [Changelog](https://github.com/moxystudio/node-cross-spawn/blob/master/CHANGELOG.md) - [Commits](moxystudio/node-cross-spawn@v7.0.3...v7.0.6) Updates `cross-spawn` from 6.0.5 to 7.0.6 - [Changelog](https://github.com/moxystudio/node-cross-spawn/blob/master/CHANGELOG.md) - [Commits](moxystudio/node-cross-spawn@v7.0.3...v7.0.6) --- updated-dependencies: - dependency-name: cross-spawn dependency-type: indirect - dependency-name: cross-spawn dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump CLI version from 1.35.1 to 1.37.4 (#1065) Co-authored-by: amitjoshi <[email protected]> * December Release Notes (#1066) * add release note * fix typo --------- Co-authored-by: Sandeep Satheesh <[email protected]> * Remove unused command from package.json (#1070) Removing the command `extension.createChatView` from package.json as it is registered but there is no handler associated with it. * Fix error type for concurrency handler (#1071) Fixes the error throws in case of bulk header rejection * Add code coverage output folder in gitignore (#1072) Adding the code coverage output folder `.nyc_ouput` folder in `.gitignore` so that it doesn't show up in `git diff` after running code coverage command. * Refactor CLI Acquisition Context (#1074) * Refactor CLI Acquisition Context * Fix import * Add tests * [PowerPages] [Create-Site] Implement multi-step site creation with enhanced input handling (#1063) * Enhance CreateSiteCommand to include extension context and add ReadonlyFileSystemProvider for site page previews * Implement EditableFileSystemProvider for site page editing and update CreateSiteHelper to utilize it * Integrate CreateSiteCommand into CommandRegistry and update related components for site creation functionality * Disable copy functionality in EditableFileSystemProvider implementation * Remove ReadonlyFileSystemProvider implementation * Add telemetry constant for previewing site pages and refactor related components * Refactor CommandRegistry and add command registration utility for chat participants * Add constants for site creation parameters and refactor NL2SiteService to use them * Refactor CreateSiteCommand and CreateSiteHelper to use structured options and improve readability; add CreateSiteTypes for better type management * Add error telemetry constant for previewing site pages and handle errors in previewSitePagesContent function * Rename fileContentMap to _fileContentMap for consistency and clarity in EditableFileSystemProvider * Implement multi-step input for site creation and register command for user inputs in CreateSiteHelper * Enhance copilot availability checks and update response structure in ArtemisService * Add site creation inputs and environment info interfaces; refactor CreateSiteCommand * Update src/common/chat-participants/powerpages/commands/create-site/CreateSiteTypes.ts Co-authored-by: Priyanshu Agrawal <[email protected]> * Update src/common/chat-participants/powerpages/commands/create-site/CreateSiteHelper.ts Co-authored-by: Priyanshu Agrawal <[email protected]> * Update src/common/chat-participants/powerpages/commands/create-site/CreateSiteHelper.ts Co-authored-by: Priyanshu Agrawal <[email protected]> * Add eslint directives to suppress no-explicit-any warnings in site page handling --------- Co-authored-by: amitjoshi <[email protected]> Co-authored-by: Priyanshu Agrawal <[email protected]> * Fix accessibility issue in Pages Copilot pane (#1076) Fix accessibility issue in `Send` button in Power Pages Copilot pane. Updated the `aria-label` to `Send` instead of `Match Case` * Fix casing of OrgUrl to orgUrl in site creation input handling (#1077) Co-authored-by: amitjoshi <[email protected]> * Add test scripts in .vscode\tasks.json (#1078) To be able to easily run tests from VS Code, added test scripts in `tasks.json` * Remove redundant step from gulp file (#1079) The step `testWeb` is redundant as the step `testUnitTests` already includes the test pattern defined in `testWeb` step. Hence, removing it from gulp config. * Added site runtime preview code behind ECS Config (#1052) * Added site runtime preview code behind ECS Config * localization added and moved preview site code to separate file * fixed build failures and handled empty website recordID * Corrected website details ppapi service for runtime preview * merged main branch to runtimePreviewECS * correcting function call for runtime preview * corrected localised error * removed bug of no show command found * Launch Edge new tab instead of launching project --------- Co-authored-by: Ashwani Kumar <[email protected]> Co-authored-by: Priyanshu Agrawal <[email protected]> * Hide preview site command from palette when feature is disabled (#1080) * Enhance feedback interaction by adding keyboard accessibility for thumbs up/down buttons (#1082) Co-authored-by: amitjoshi <[email protected]> * Add conditional visibility for Current Active Users command in web virtual workspace and remove alt cmds (#1083) Co-authored-by: amitjoshi <[email protected]> * Handle null language code for content snippets and update query parameters in portal schema (#1086) Co-authored-by: amitjoshi <[email protected]> * Enhance accessibility by adding ARIA attributes and keyboard navigation support (#1085) Co-authored-by: amitjoshi <[email protected]> * Enhance accessibility by adding ARIA attributes and making thinking message focusable (#1087) Co-authored-by: amitjoshi <[email protected]> * Add sovereign cloud endpoints for PPAPI service (#1081) * Site Preview Enhancements: Clear cache and telemetry logging (#1088) * Enhance accessibility by improving focus styles and adding ARIA attributes in notification panel (#1089) Co-authored-by: amitjoshi <[email protected]> * Actions Hub: Implement base classes to show tree structure (#1090) * Add models for Actions Hub * Basic wireup * Add models * ToDo * Remove foo environment * Remove error * Refactor * Add translations * Add tests * Add tree item for No sites found * Add translations * Added changes for orgGeo (#1093) * Refactor: Move actions-hub and preview-site folder under power-pages (#1092) * Refactor: Move actions-hub and preview-site folder under power-pages * Remove unwanted file * Refactor * Add PPAPI to known words * Disable actions hub by default * Web Extension: Remove dependency on website preview URL (#1091) * Web Extension: Remove dependency on website preview URL * Translations * Web Extension: Fix file opening for media files (#1094) Using `vscode.open` API to open the default file instead of `window.showTextDocument` because `showTextDocument` doesn't open binary files to it was failing to open non-text files. * Bump CLI version from 1.37.4 to 1.39.3 (#1095) * Fix environment icon for dark mode (#1097) * Fix environment icon for dark mode * Refactor * Fix test * fix screen reader issue in copilot chat (#1100) Co-authored-by: Sandeep Satheesh <[email protected]> * add focus ring (#1099) Co-authored-by: Sandeep Satheesh <[email protected]> * add release notes (#1101) Co-authored-by: Sandeep Satheesh <[email protected]> * Site Preview: Initialize after ECS fetch (#1102) * Site Preview: Initialize after ECS fetch * Set context after loading website URL * Add source attribute handling and telemetry for invalid attributes in web extension (#1105) * Add source attribute handling and telemetry for invalid attributes in web extension * Refactor source attribute handling in processDataAndCreateFile function --------- Co-authored-by: amitjoshi <[email protected]> * Enhance Actions Hub with environment support and unit tests (#1098) * Enhance Actions Hub: Add support for environment retrieval and localization updates * Add unit tests for ActionsHubTreeDataProvider functionality --------- Co-authored-by: amitjoshi <[email protected]> * Fix A11y bug (#1108) * Actions Hub: Add toggle based on ECS (#1107) * Add feature flag for Actions Hub * Refactor * Fix errors * Fix errors * Add unit tests * [PowerPages] [Site Preview] Simplify preview functionality (#1109) * Refactor telemetry handling by removing ITelemetry parameter from functions (#1110) * Refactor telemetry handling by removing ITelemetry parameter from sendTelemetryEvent and related functions * Refactor: Remove telemetry parameter from getEndpoint and related functions * Refactor: Remove telemetry parameter from authentication functions and related calls * Refactor: Remove telemetry parameter from getArtemisResponse call in activate function * Refactor: Remove telemetry parameter from various authentication and service calls --------- Co-authored-by: amitjoshi <[email protected]> * Webpack: Ignore warnings (#1113) * Run desktop integration tests in CI (#1114) * Run desktop integration tests in CI * Run desktop test only on Windows * Enhance Actions Hub with environment retrieval and refresh functionality (#1111) * Enhance Actions Hub: Add support for environment retrieval and localization updates * Add unit tests for ActionsHubTreeDataProvider functionality * Add AuthManager class and AuthInfo interface for authentication handling * Enhance authentication handling: Add auth info extraction and update environment info retrieval in Actions Hub * Add refresh command and event handling for Actions Hub environment changes * Refactor ActionsHubTreeDataProvider: Manual refresh should trigger Pac * Remove AuthManager class and related authentication logic * Add error handling to refresh command in ActionsHubTreeDataProvider * RemoveDependencyOnOldTelemetryCluster * Refactor ActionsHubTreeDataProvider tests to include PacTerminal in initialization * Remove telemetry dependency from intelligence API endpoint retrieval * Refactor telemetry handling by removing ITelemetry parameter from sendTelemetryEvent and related functions * Refactor: Remove telemetry parameter from getEndpoint and related functions * Refactor: Remove telemetry parameter from authentication functions and related calls * Refactor telemetry event assertion in AuthenticationProvider tests * Refactor: Remove telemetry parameter from getArtemisResponse call in activate function * Refactor: Remove telemetry parameter from various authentication and service calls * Refactor: Rename AuthManager to PacAuthManager for consistency * Refactor: Use pacWrapper for activeAuth retrieval in ActionsHubTreeDataProvider * Add ACTIONS_HUB_REFRESH_FAILED constant and update error logging in ActionsHubTreeDataProvider * Remove unused AuthInfo interface from Constants.ts * Refactor ActionsHub initialization by removing authentication handling and simplifying the process * Add localization for expired environment message and update refresh title --------- Co-authored-by: amitjoshi <[email protected]> * Remove old telemetry usage (#1112) * Remove old telemetry usage * Remove more usage * Remove more usage * Remove more usage * Remove more usage * Fix tests * Fix typo * Comment tests * Refactor Actions Hub and Preview Site initialization logic (#1115) * [PowerPages][Actions Hub] Toggle to Change Environment (#1119) * Add switch environment functionality to Actions Hub * Update icon for switch environment command in Actions Hub * Update translation for switch environment title to "Change Environment" * Add tests for switch environment command registration and error handling * Remove redundant error handling tests for switch environment and refresh commands * Refactor command names to include namespace for actions in Actions Hub * Update refresh command registration to include namespace in Actions Hub tests --------- Co-authored-by: amitjoshi <[email protected]> * Bump serialize-javascript, gulp-mocha and mocha (#1120) Bumps [serialize-javascript](https://github.com/yahoo/serialize-javascript) to 6.0.2 and updates ancestor dependencies [serialize-javascript](https://github.com/yahoo/serialize-javascript), [gulp-mocha](https://github.com/sindresorhus/gulp-mocha) and [mocha](https://github.com/mochajs/mocha). These dependencies need to be updated together. Updates `serialize-javascript` from 5.0.1 to 6.0.2 - [Release notes](https://github.com/yahoo/serialize-javascript/releases) - [Commits](yahoo/serialize-javascript@v5.0.1...v6.0.2) Updates `gulp-mocha` from 8.0.0 to 10.0.1 - [Release notes](https://github.com/sindresorhus/gulp-mocha/releases) - [Commits](sindresorhus/gulp-mocha@v8.0.0...v10.0.1) Updates `mocha` from 9.2.2 to 11.1.0 - [Release notes](https://github.com/mochajs/mocha/releases) - [Changelog](https://github.com/mochajs/mocha/blob/main/CHANGELOG.md) - [Commits](mochajs/mocha@v9.2.2...v11.1.0) --- updated-dependencies: - dependency-name: serialize-javascript dependency-type: indirect - dependency-name: gulp-mocha dependency-type: direct:development - dependency-name: mocha dependency-type: direct:development ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * [Power Pages][Actions Hub] Add "Show Environment Details" to Actions Hub (#1118) * [Power Pages][Actions Hub] Add "Show Environment Details" to Actions Hub * [PowerPages][Actions Hub] Update icon for switch environment command * [PowerPages][Actions Hub] Remove refresh command from Actions Hub navigation * Fix order * Update translation identifiers and fix newline in localization files * Refactor Actions Hub commands and extract command handlers into a new file * Add tests for switchEnvironment command * [PowerPages][ActionsHub] - Show Active and Inactive Sites (#1117) * show active and inactive sites in actions hub view * add integration tests for actions hub * [Power Pages][Actions Hub] Add NoSitesTreeItem to handle empty site groups * [Power Pages][Actions Hub] Update website details property names to PascalCase and change collapsible state to expanded * Refactor environment handling: rename method for clarity, improve error handling, and streamline environment selection process * Update localization strings for environment selection and session details * Refactor authentication and organization data structures; update TypeScript configuration and remove unused PacAuthManager * Add ArtemisContext for managing service responses; update ActionsHub and related components to utilize new context --------- Co-authored-by: Sandeep Satheesh <[email protected]> Co-authored-by: Priyanshu Agrawal <[email protected]> * [PowerPages][Actions Hub] Add Instance URL to Environment Details (#1122) * Add 'isCurrent' property and description to site tree items (#1123) * Update ArtemisService URLs to use production segment for GCC, high, mooncake, and DOD environments (#1121) Co-authored-by: amitjoshi <[email protected]> * [PowerPages][Actions Hub] Add commands to open active and inactive site in PP Studio (#1124) * [PowerPages][Actions Hub] Add commands to open active and inactive sites in Power Pages Studio * Refactor studio URL handling to use constants for endpoints * Add localization entries for "Current" and "Instance url" in JSON and XLF files * Refactor studio URL generation to include environment ID and add openInactiveSitesInStudio function * [Power Pages][Actions Hub] Add ability to preview active sites in VS Code (#1127) * Add preview command and siteInfo property to SiteTreeItem * Add unit tests for PreviewSite.isSiteRuntimePreviewEnabled method * Refactor PreviewSite initialization and loadSiteUrl method; update tests for loadSiteUrl functionality * Add tests for ActionsHubTreeDataProvider.getChildren method; mock website data for active and inactive sites * Add mocks for active and all websites in ActionsHubTreeDataProvider tests * Refactor site context values to distinguish between current and non-current active sites; update related tests * Add websiteId property to IWebsiteInfo interface and update related tree item implementations * Update walkthrough title to use h4 and enhance focus indicator for accessibility (#1125) Co-authored-by: amitjoshi <[email protected]> * Add new command to create authentication profile and handle errors in actions hub (#1128) * Add new command to create authentication profile and handle errors in actions hub * Refactor tests for ActionsHubCommandHandlers and ActionsHubTreeDataProvider to improve argument validation and remove unnecessary assertions * Stub AuthInfo and OrgInfo getters in ActionsHubTreeDataProvider tests to improve test reliability * Remove redundant test for creating new auth profile in ActionsHubCommandHandlers * Fix async initialization in ActionsHub and streamline environment info handling in ActionsHubTreeDataProvider * Implement createAuthProfileExp function and add tests for authentication profile creation * Refactor createAuthProfileExp function import path and add new utility file for authentication profile creation * Add tests for ActionsHubTreeDataProvider to validate site retrieval and error handling * Remove redundant tests from ActionsHubTreeDataProvider for getChildren method * Refactor import path for createAuthProfileExp to new utility file in PacAuthUtil * Remove tests for PacAuthUtils as part of code cleanup * Add comment to clarify DV authentication requirement in createNewAuthProfile --------- Co-authored-by: amitjoshi <[email protected]> * [Power Pages][Actions Hub] Show current site (#1129) * Refactor site context handling and update site data types for improved clarity * Refactor website data handling to use IWebsiteDetails and streamline site fetching logic * Refactor import statements in ActiveGroupTreeItem, EnvironmentGroupTreeItem, and InactiveGroupTreeItem for improved organization and clarity * Refactor import statements in InactiveGroupTreeItem for improved organization * Add tests for PacContext and improve context update logic * Add tests for previewSite functionality in ActionsHubCommandHandlers * Add fetchWebsites function and related tests for website retrieval logic * Add tests for ActiveGroupTreeItem and update siteInfo assertions * Add tests for ActionsHubTreeDataProvider to verify website loading logic * Add websiteid to VSCode settings for improved configuration * Refactor ActionsHubTreeDataProvider to simplify loadWebsites method and improve refresh logic * Refactor CurrentSiteContext to utilize utility functions for website YAML handling and improve code clarity * Refactor ActionsHubTreeDataProvider to streamline website loading logic and improve refresh handling * [Power Pages][Actions Hub] Update "Change environment" option to match Figma (#1130) * [Power Pages][Actions Hub] Update "Change environment" option to match Figma * Add test for environment switching in Actions Hub command handlers * Refactor environment switching to use detailed description and enhance test coverage for environment selection * Refactor tests for ActionsHubTreeDataProvider by removing redundant cases and ensuring proper stubbing of fetchWebsites * Disable website loading in ActionsHubTreeDataProvider tests to ensure empty result for specific scenarios * [Power Pages][Actions Hub] Add revealInOS command for current active site and update translations (#1131) * Refactor website details to use camelCase properties and add siteVisibility field (#1133) * [Power Pages][Actions Hub] Update command references for authentication profile (#1132) * [Power Pages][Actions Hub] Update command references for authentication profile * Update command reference for new authentication profile in ActionsHubTreeDataProvider test --------- Co-authored-by: amitjoshi <[email protected]> Co-authored-by: Priyanshu Agrawal <[email protected]> Co-authored-by: Priyanshu Agrawal <[email protected]> * [Power Pages][Actions Hub] Add open site management functionality (#1136) * [Power Pages][Actions Hub] Add open site management functionality * Update translation for "Open Site Management" to use lowercase in localization files * [PowerPages][ActionsHub] Upload site command with visibility confirmation (#1134) * Refactor website details to use camelCase properties and add siteVisibility field * [Power Pages][Actions Hub] Add upload site command with confirmation dialog * [Power Pages][Actions Hub] Implement upload site command with confirmation dialog and tests * [Power Pages][Actions Hub] Add site visibility check before upload confirmation dialog * [Power Pages][Actions Hub] Enhance upload site command to handle public and private site visibility * Update login message links and adjust Actions Hub feature gate settings * [Power Pages][Actions Hub] Update upload site tests to use warning message and reset stubs * Refactor upload site tests to use showInformationMessage and remove unnecessary stubs * Refactor uploadSite tests to use showInformationMessage and handle site visibility for public and private sites * Refactor uploadSite test to remove unused showInformationMessage stub * ``` Refactor uploadSite tests to improve clarity and maintainability ``` * Stub showInformationMessage in ActionsHubCommandHandlers tests for improved test isolation * Refactor ActionsHubCommandHandlers tests to improve test isolation by removing unused stubs and enhancing setup * Refactor test setup in ActionsHubCommandHandlers for improved isolation --------- Co-authored-by: Priyanshu Agrawal <[email protected]> Co-authored-by: amitjoshi <[email protected]> * Update login command reference and add upload site title in localization file (#1138) Co-authored-by: amitjoshi <[email protected]> * [Power Pages] Add title for Copilot feedback Submit button (#1139) * [PowerPages] [Actions Hub] Add support for other sites in Actions Hub (#1140) * Add support for other sites in Actions Hub and update related interfaces * Update tests for OtherSitesGroupTreeItem and ActionsHubTreeDataProvider to include handling of other sites * Refactor findOtherSites function to accept fs and yaml modules as parameters for improved testability * Add mockFs and mockYaml to ActionsHubCommandHandlers tests for improved isolation * Remove test for YAML parsing errors in ActionsHubCommandHandlers to streamline error handling * Disable Actions Hub Panel by default in fallback configuration * Clarify comment on dataModelVersion in OtherSitesGroupTreeItem to specify defaulting behavior for other sites * Add js-yaml dependency and update YAML parsing in WorkspaceInfoFinderUtil * Remove yaml dependency from package.json and package-lock.json * Replace js-yaml with yaml package and update YAML parsing methods * Refactor findOtherSites and createKnownSiteIdsSet to use Set for known site IDs * Refactor findOtherSites and createKnownSiteIdsMap to use Set for known site IDs --------- Co-authored-by: amitjoshi <[email protected]> * [Power Pages][Actions Hub] Add site details command (#1141) * [Power Pages][Actions Hub] Add site details command * Add openSiteManagement function with error handling and tests * Fix menu item order * [Power Pages] [Actions Hub] Add download site command to Actions Hub (#1143) * [Power Pages][Actions Hub] Add download site command * Add tests * [PowerPages][ActionsHub] Upload functionality for other sites (#1144) * [Power Pages] Enhance upload functionality to support other sites in Actions Hub * [Power Pages] Enhance uploadSite functionality with confirmation handling and error management * [Power Pages][Actions Hub] Refactor test setup by removing unused stubs in ActionsHubCommandHandlers tests * [Power Pages][Actions Hub] Remove unused error handling test for uploadSite and update data model version for other sites * [Power Pages][Actions Hub] Remove unused test for upload handling of other sites in ActionsHubCommandHandlers --------- Co-authored-by: amitjoshi <[email protected]> * [Power Pages] [Actions Hub] Add 'Open in Power Pages Studio' command (#1145) * [Power Pages][Preview Site] Show notification when previewing site with pending changes (#1146) * [Power Pages][Preview Site] Show notification when previewing site with pending changes * [Power Pages][Localization] Update localization strings for site preview notifications * [Power Pages][Preview Site] Enhance site preview functionality with site visibility and upload path adjustments * [Power Pages][PacInterop] Update PAC executable path to use dynamic CLI path * Refactor uploadSite function calls to include an empty string parameter for consistency * Refactor uploadSite to use logical OR for websitePath fallback * [Power Pages] [Actions Hub] Enhance revealInOS command to support other sites (#1147) * [Power Pages] [Actions Hub] Initialize OrgChangeNotifier instance in activate function for all workspaces (#1148) Co-authored-by: amitjoshi <[email protected]> * [Power Pages] [Actions Hub] Handle null or undefined labels in ActionsHubTreeItem (#1149) * [Power Pages] [Actions Hub] Handle null or undefined labels in ActionsHubTreeItem * [Power Pages] [Actions Hub] Refactor isCurrent logic and enhance tests for ActiveGroupTreeItem * Fix typos and improve file path handling in tests (#1151) - ✏️ Corrected spelling of "Pattern" in test names - 🔧 Updated file path handling to use vscode.Uri.file - 📂 Simplified file path construction for better readability -Priyanshu * Remove outline from feedback button styles (#1150) - ✨ Removed the outline property from button styles for improved accessibility. -Priyanshu * [Power Pages] [Copilot] Add localization for Copilot response and code block messages; enhance accessibility with live regions for screen readers (#1152) Co-authored-by: amitjoshi <[email protected]> * Fix typos in OneDSLogger and oneDSLoggerWrapper (#1153) - ✏️ Corrected spelling of 'initialize' in OneDSLogger and oneDSLoggerWrapper classes - 🔧 Updated variable name from 'oneDSLoggerIntance' to 'oneDSLoggerInstance' for consistency -Priyanshu * Enhance website management features (#1155) - ✨ Added new properties `createdOn` and `creator` to website details. - 🔄 Refactored site visibility handling to use an enum. - 🛠️ Updated various components to utilize new properties. - ✅ Enhanced tests to cover new functionality. -Priyanshu * [Powerpages][ActionsHub]Sort active websites by creation date in fetchWebsites function (#1156) * Sort active websites by creation date in fetchWebsites function * Refactor website sorting logic to use a dedicated function for sorting by creation date --------- Co-authored-by: amitjoshi <[email protected]> * [Power Pages][Actions Hub] Add unit tests (#1157) * Add unit tests for getActiveWebsites function - 🧪 Implement tests for calling PPAPIService.getAllWebsiteDetails with correct parameters. - ✅ Validate return values from PPAPIService for active websites. - 🛠️ Handle empty array responses from PPAPIService. - 🚨 Ensure errors from PPAPIService are propagated correctly. -Priyanshu * Update getAppModules to accept accessToken parameter - 🔧 Modified getAppModules function to accept accessToken as a parameter. - 🔄 Updated getAllWebsites to pass accessToken when calling getAppModules. - 📜 Adjusted related API calls to ensure proper token usage. -Priyanshu * Add coding and test guidelines to documentation (#1158) This commit introduces a comprehensive set of coding and test instructions to ensure consistency and quality in the codebase. - 📜 Added coding guidelines including naming conventions and style rules. - 🧪 Included test instructions for writing and running tests effectively. -Priyanshu * Bump CLI version to 1.40.4 (#1159) Bump CLI version to latest * [Power Pages][Actions Hub] Enhance telemetry logging and error handling (#1161) * Enhance telemetry logging and error handling - ✨ Added `getBaseEventInfo` function for enriched telemetry data. - 🔧 Updated logging in `ActionsHub`, `ActionsHubCommandHandlers`, and `ActionsHubTreeDataProvider` to include event info. - 🐛 Improved error handling in `fetchWebsites` and `loadWebsites` methods. - ✅ Added unit tests for `getBaseEventInfo` and updated existing tests for logging changes. -Priyanshu * Enhance telemetry and error handling - ✨ Added OS detection in telemetry events. - 🔧 Improved logging for environment switching and site management actions. - 🐛 Fixed error handling in various command handlers. - 📜 Updated constants for new telemetry events. - 🧪 Added tests for telemetry helper functions. -Priyanshu * Refactor telemetry test for clarity - 🧹 Remove unused OS import from TelemetryHelper.test.ts - 🔄 Rename 'siteId' to 'currentSiteId' for consistency in event info - ✅ Update test assertions to reflect the new naming - 🧪 Ensure tests maintain expected behavior after changes -Priyanshu * Add tenantId to event info and update tests - ✨ Added tenantId to the event info in getBaseEventInfo. - 🔧 Updated tests to include tenantId in various scenarios. - 🧪 Enhanced test stubs for AuthInfo to ensure accurate testing. - 📜 Improved test coverage for cases with undefined values. - 🛠️ Fixed formatting issues in TelemetryHelper.ts. -Priyanshu * Enhance telemetry and localization support - ✨ Added new localization strings for environment changes, site visibility, creator, and created date. - 🔧 Refactored telemetry helper to remove OS dependency and improve event info structure. - 🧪 Updated tests to reflect changes in telemetry event info and removed OS checks. -Priyanshu * Close existing Edge DevTools before launching preview (#1163) - 🔧 Added a method to close existing Edge DevTools tabs before launching a new preview. - 🚀 Updated the site preview command to ensure no duplicate DevTools instances are open. -Priyanshu * Add preview message for published changes (#1164) - ✨ Added a new message for the preview indicating that it shows published changes. - 🔧 Updated localization files to include the new message. - 📜 Modified the Constants and PreviewSite files to utilize the new message. -Priyanshu --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: amitjoshi438 <[email protected]> Co-authored-by: amitjoshi <[email protected]> Co-authored-by: Ashwani Kumar <[email protected]> Co-authored-by: Ashwani Kumar <[email protected]> Co-authored-by: tyaginidhi <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Sandeep Satheesh <[email protected]> Co-authored-by: Sandeep Satheesh <[email protected]> Co-authored-by: BidishaMS <[email protected]>
1 parent d6d2968 commit 2b7ec7c

30 files changed

+1427
-238
lines changed

.github/copilot-instructions.md

Lines changed: 66 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,66 @@
1+
# Instructions
2+
3+
## Coding guidelines
4+
5+
These are the coding guidelines which are used in this repository. Always follow these guidelines when generating code..
6+
7+
### Indentation
8+
9+
We use spaces, not tabs. Indentation level is 4 spaces.
10+
11+
### Naming Conventions
12+
13+
* Use PascalCase for `type` names
14+
* Use PascalCase for `enum` values
15+
* Use camelCase for `function` and `method` names
16+
* Use camelCase for `property` names and `local variables`
17+
* Use whole words in names when possible
18+
* Use UPPER_CASE for constants.
19+
20+
### Comments
21+
22+
* When there are comments for `functions`, `interfaces`, `enums`, and `classes` use JSDoc style comments
23+
24+
### Strings
25+
26+
* Use "double quotes" for strings shown to the user that need to be externalized (localized)
27+
* Use 'single quotes' otherwise
28+
* All strings visible to the user need to be externalized
29+
30+
### Style
31+
32+
* Use arrow functions `=>` over anonymous function expressions
33+
* Only surround arrow function parameters when necessary. For example, `(x) => x + x` is wrong but the following are correct:
34+
35+
```javascript
36+
x => x + x
37+
(x, y) => x + y
38+
<T>(x: T, y: T) => x === y
39+
```
40+
41+
* Always surround loop and conditional bodies with curly braces
42+
* Open curly braces always go on the same line as whatever necessitates them
43+
* Parenthesized constructs should have no surrounding whitespace. A single space follows commas, colons, and semicolons in those constructs. For example:
44+
45+
```javascript
46+
for (let i = 0, n = str.length; i < 10; i++) {
47+
if (x < 10) {
48+
foo();
49+
}
50+
}
51+
52+
function f(x: number, y: string): void { }
53+
```
54+
55+
## Test instructions
56+
57+
Follow below along with [Coding Guidelines](#coding-guidelines) when generating tests.
58+
59+
* Use `expect` from `chai` for writing assertions.
60+
* Generate mocha style tests with `describe` and `it` blocks.
61+
* Use `sinon` for creating stubs and spies.
62+
* Mock dependencies as much as possible when writing tests for a particular function.
63+
* To run tests inside `client/test/integration` folder, you can ask to run `npm run test-desktop-int` command.
64+
* To run tests inside `web/test/integration` folder, you can ask to run `npm run test-web-integration` command.
65+
* Write tests to cover all possible scenarios.
66+
* Do not write the comments //Arrange, //Act and //Assert before code.

.vscode/settings.json

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,14 @@
11
{
22
"cSpell.words": [
33
"appmoduleid",
4+
"createdon",
45
"dataverse",
56
"entityrecord",
67
"Entra",
78
"lcid",
89
"MSPP",
910
"nupkg",
11+
"owninguser",
1012
"pagetype",
1113
"powerapps",
1214
"powerpages",

gulpfile.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -339,7 +339,7 @@ async function snapshot() {
339339
}
340340
}
341341

342-
const cliVersion = '1.39.3';
342+
const cliVersion = '1.40.4';
343343

344344
const recompile = gulp.series(
345345
clean,

l10n/bundle.l10n.json

Lines changed: 31 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -133,6 +133,7 @@
133133
"Unable to clear cache": "Unable to clear cache",
134134
"Your preview isn't updated. Please upload your site to see the latest changes.": "Your preview isn't updated. Please upload your site to see the latest changes.",
135135
"Upload changes": "Upload changes",
136+
"The preview shown is for published changes. Please publish any pending changes to see them in the preview.": "The preview shown is for published changes. Please publish any pending changes to see them in the preview.",
136137
"Enter the name of the web template": "Enter the name of the web template",
137138
"Please enter a name for the web template.": "Please enter a name for the web template.",
138139
"A webtemplate with the same name already exists. Please enter a different name.": "A webtemplate with the same name already exists. Please enter a different name.",
@@ -204,6 +205,7 @@
204205
"Browse...": "Browse...",
205206
"Select the folder that will contain your project root for your site": "Select the folder that will contain your project root for your site",
206207
"Select Folder": "Select Folder",
208+
"Environment changed successfully.": "Environment changed successfully.",
207209
"Timestamp: {0}/{0} is the timestamp": {
208210
"message": "Timestamp: {0}",
209211
"comment": [
@@ -270,18 +272,42 @@
270272
"{0} is the website name"
271273
]
272274
},
273-
"Website ID: {0}/{0} is the website ID": {
274-
"message": "Website ID: {0}",
275+
"Website Id: {0}/{0} is the website ID": {
276+
"message": "Website Id: {0}",
275277
"comment": [
276278
"{0} is the website ID"
277279
]
278280
},
279-
"Data model version: v{0}/{0} is the data model version": {
280-
"message": "Data model version: v{0}",
281+
"Data model version: {0}/{0} is the data model version": {
282+
"message": "Data model version: {0}",
281283
"comment": [
282284
"{0} is the data model version"
283285
]
284286
},
287+
"Website Url: {0}/{0} is the website Url": {
288+
"message": "Website Url: {0}",
289+
"comment": [
290+
"{0} is the website Url"
291+
]
292+
},
293+
"Site visibility: {0}/{0} is the site visibility": {
294+
"message": "Site visibility: {0}",
295+
"comment": [
296+
"{0} is the site visibility"
297+
]
298+
},
299+
"Creator: {0}/{0} is the creator": {
300+
"message": "Creator: {0}",
301+
"comment": [
302+
"{0} is the creator"
303+
]
304+
},
305+
"Created on: {0}/{0} is the created date": {
306+
"message": "Created on: {0}",
307+
"comment": [
308+
"{0} is the created date"
309+
]
310+
},
285311
"PAC Telemetry enabled": "PAC Telemetry enabled",
286312
"Failed to enable PAC telemetry.": "Failed to enable PAC telemetry.",
287313
"PAC Telemetry disabled": "PAC Telemetry disabled",
@@ -387,4 +413,4 @@
387413
"The {0} represents profile's Azure Cloud Instances"
388414
]
389415
}
390-
}
416+
}

loc/translations-export/vscode-powerplatform.xlf

Lines changed: 39 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -144,18 +144,26 @@ The second line should be '[TRANSLATION HERE](https://aka.ms/pages-clear-cache).
144144
<trans-unit id="++CODE++b749e205302cc21952a1e9646569ca0af9017cc5f39dd741830551bdd4ae823e">
145145
<source xml:lang="en">Copy to clipboard</source>
146146
</trans-unit>
147+
<trans-unit id="++CODE++a065334fc8846953a2dacf5096572bb814db18ac32fdb3e22cff0d387cdd4393">
148+
<source xml:lang="en">Created on: {0}</source>
149+
<note>{0} is the created date</note>
150+
</trans-unit>
147151
<trans-unit id="++CODE++446092146c92f1aa2441a29971d3844b0add13a951c45c491423f506bbe4edab">
148152
<source xml:lang="en">Creating {0}...</source>
149153
<note>{0} will be replaced by the entity type.</note>
150154
</trans-unit>
155+
<trans-unit id="++CODE++ec8fd606d46114985d2f628dc67992093b8abb32489b3cbbfbb86bba96925943">
156+
<source xml:lang="en">Creator: {0}</source>
157+
<note>{0} is the creator</note>
158+
</trans-unit>
151159
<trans-unit id="++CODE++e0d1b68224bf0b31ef16b206c65b5f8f6b89d18161f4a7cdcb8d0ac8d952549a">
152160
<source xml:lang="en">Current</source>
153161
</trans-unit>
154162
<trans-unit id="++CODE++fc47ac56c8a4446c02d8470168a888d8e0040cfc0bb5ce6173e324dccc72f24c">
155163
<source xml:lang="en">Current site path not found.</source>
156164
</trans-unit>
157-
<trans-unit id="++CODE++1da89bd5f5df020ca47c79dfdbc1517a299ca77570df2de4132f53797d5f2412">
158-
<source xml:lang="en">Data model version: v{0}</source>
165+
<trans-unit id="++CODE++e159063579c9e875ce444b930e8feb5d4fc1a93af6acb47d7a5cf9d5c5241153">
166+
<source xml:lang="en">Data model version: {0}</source>
159167
<note>{0} is the data model version</note>
160168
</trans-unit>
161169
<trans-unit id="++CODE++5bd78fcb75f98d65ab8f89a8f0b6681c25c13308fa8643c44ab459b7783c06c9">
@@ -198,6 +206,9 @@ The {3} represents Solution's Type (Managed or Unmanaged), but that test is loca
198206
<source xml:lang="en">Environment ID: {0}</source>
199207
<note>{0} is the Environment ID</note>
200208
</trans-unit>
209+
<trans-unit id="++CODE++273ac70fb32a6f135f9998bf3601f83596fccea343ed551d2a405c4958486720">
210+
<source xml:lang="en">Environment changed successfully.</source>
211+
</trans-unit>
201212
<trans-unit id="++CODE++5a2391041cb1f6f5ab0cce07946dbf4222358fbd947768d94ac12ef5329485d3">
202213
<source xml:lang="en">Explain the following code snippet:</source>
203214
</trans-unit>
@@ -513,6 +524,10 @@ The {3} represents Dataverse Environment's Organization ID (GUID)</note>
513524
<trans-unit id="++CODE++f7094162bd4c4b59aa706387714347223e1569c79c5b37eb19ac754a33a0fdd0">
514525
<source xml:lang="en">Site runtime preview feature is not enabled.</source>
515526
</trans-unit>
527+
<trans-unit id="++CODE++a2eb2f1a32eedafa3e666685f09ee073175d309d6137b80edac517b51bffd84d">
528+
<source xml:lang="en">Site visibility: {0}</source>
529+
<note>{0} is the site visibility</note>
530+
</trans-unit>
516531
<trans-unit id="++CODE++27dd8c8e67d15783f51c6b71ed60d5350731daa85b418f1e84c2e5828ae2d54e">
517532
<source xml:lang="en">Some references might be broken. Please check diagnostics for details.</source>
518533
</trans-unit>
@@ -542,8 +557,8 @@ The {3} represents Dataverse Environment's Organization ID (GUID)</note>
542557
<trans-unit id="++CODE++5eccedc8c04d2137a9eb74edf18ea469b4b63e7315de4b8fe49c48bb8d0780e9">
543558
<source xml:lang="en">The pac CLI is ready for use in your VS Code terminal!</source>
544559
</trans-unit>
545-
<trans-unit id="++CODE++db00163e0e6fed2e319e28785c537befb88ded272d245e503d9580322ab2fec8">
546-
<source xml:lang="en">The preview shown is for published changes.</source>
560+
<trans-unit id="++CODE++51d2fcbba3e5fc15afbd85f1d179e58b9c4328b466cd7872ec77dc0c9cb34afe">
561+
<source xml:lang="en">The preview shown is for published changes. Please publish any pending changes to see them in the preview.</source>
547562
</trans-unit>
548563
<trans-unit id="++CODE++18f3be2933741b96e56b1e1f70818fcba46ec9c5b3dcd0e28d801eab15791d84">
549564
<source xml:lang="en">There was a permissions problem with the server</source>
@@ -611,10 +626,14 @@ The {3} represents Dataverse Environment's Organization ID (GUID)</note>
611626
<trans-unit id="++CODE++00dc171124ab430bbbaae51ec39dda1c5e7d045f382f56b1767d9e733292731c">
612627
<source xml:lang="en">Webpage names should contain only letters, numbers, hyphens, or underscores.</source>
613628
</trans-unit>
614-
<trans-unit id="++CODE++091327257429303159b8580005b5b5a1e05fd028030817edaaa835cfcb0b3293">
615-
<source xml:lang="en">Website ID: {0}</source>
629+
<trans-unit id="++CODE++d3eaa7bf5377b15781ad4c0f43e4da94b9a2bd485ddbbbe62a2c3d53c5c7fe4a">
630+
<source xml:lang="en">Website Id: {0}</source>
616631
<note>{0} is the website ID</note>
617632
</trans-unit>
633+
<trans-unit id="++CODE++4f5909ac007d8a897ec18ede71cdf8f6b4af3bbe78bae2f13871d50a3cebb3f6">
634+
<source xml:lang="en">Website Url: {0}</source>
635+
<note>{0} is the website Url</note>
636+
</trans-unit>
618637
<trans-unit id="++CODE++aa64cfbb63d3382bc0530c6e599467b1cc5c1ef8ff3b573d37ad628f9058b2aa">
619638
<source xml:lang="en">Website not found in the environment. Please check the credentials and login with correct account.</source>
620639
</trans-unit>
@@ -668,8 +687,8 @@ The {3} represents Dataverse Environment's Organization ID (GUID)</note>
668687
<source xml:lang="en">Auth Profiles</source>
669688
</trans-unit>
670689
<trans-unit id="microsoft-powerapps-portals.walkthrough.saveConflict.description">
671-
<source xml:lang="en">Avoid accidental overwrites when you try to save outdated code in VS Code for the Web. You can compare the changes side-by-side and decide to accept or revert the changes.
672-
690+
<source xml:lang="en">Avoid accidental overwrites when you try to save outdated code in VS Code for the Web. You can compare the changes side-by-side and decide to accept or revert the changes.
691+
673692
To learn more, visit [Prevent accidental overwrites](command:powerplatform-walkthrough.saveConflict-learn-more).</source>
674693
<note>This is a Markdown formatted string, and the formatting must persist across translations.
675694
The fifth line should be '[TRANSLATION HERE](command:powerplatform-walkthrough.saveConflict-learn-more).', keeping brackets and the text in the parentheses unmodified</note>
@@ -738,12 +757,12 @@ The fifth line should be '[TRANSLATION HERE](command:powerplatform-walkthrough.s
738757
<source xml:lang="en">File explorer</source>
739758
</trans-unit>
740759
<trans-unit id="microsoft-powerapps-portals.walkthrough.fileSystem.description">
741-
<source xml:lang="en">Find your page files organized in folders under your site name. All your pages are arranged into HTML, CSS, and JS files within the respective site component folders.
742-
743-
You can find site components like forms, content snippets, lists, and more to edit.
744-
745-
To learn more, visit [Power Pages file structure in VS Code](command:powerplatform-walkthrough.fileSystem-documentation).
746-
760+
<source xml:lang="en">Find your page files organized in folders under your site name. All your pages are arranged into HTML, CSS, and JS files within the respective site component folders.
761+
762+
You can find site components like forms, content snippets, lists, and more to edit.
763+
764+
To learn more, visit [Power Pages file structure in VS Code](command:powerplatform-walkthrough.fileSystem-documentation).
765+
747766
[Visit site folder](command:powerplatform-walkthrough.fileSystem-open-folder)</source>
748767
<note>This is a Markdown formatted string, and the formatting must persist across translations.
749768
The seventh line should be '[TRANSLATION HERE](command:powerplatform-walkthrough.fileSystem-documentation).', keeping brackets and the text in the parentheses unmodified
@@ -783,7 +802,7 @@ The second line should be '[TRANSLATION HERE](command:pacCLI.authPanel.newAuthPr
783802
</trans-unit>
784803
<trans-unit id="microsoft-powerapps-portals.walkthrough.overview.description">
785804
<source xml:lang="en">Now easily edit code of your Power Pages site. Access code from supported site components like forms, content snippets, lists, and more from within Visual Studio Code.
786-
805+
787806
To learn more, visit [Edit Power Pages code in VS Code](command:powerplatform-walkthrough.overview-learn-more).</source>
788807
<note>This is a Markdown formatted string, and the formatting must persist across translations.
789808
The second line should be '[TRANSLATION HERE](command:powerplatform-walkthrough.overview-learn-more)', keeping brackets and the text in the parentheses unmodified</note>
@@ -878,11 +897,11 @@ The second line should be '[TRANSLATION HERE](command:powerplatform-walkthrough.
878897
</trans-unit>
879898
<trans-unit id="microsoft-powerapps-portals.walkthrough.advancedCapabilities.description">
880899
<source xml:lang="en">Visual Studio Code for Web enables editing and publishing of web pages on your website.
881-
882-
For a command line interface and more advanced capabilities, install the Power Platform Extension for VS Code, available in the VS Code Marketplace for desktop.
883-
900+
901+
For a command line interface and more advanced capabilities, install the Power Platform Extension for VS Code, available in the VS Code Marketplace for desktop.
902+
884903
[Learn More](command:powerplatform-walkthrough.advancedCapabilities-learn-more) about the difference between Visual Studio Code for desktop and web.
885-
904+
886905
[Start coding](command:powerplatform-walkthrough.advancedCapabilities-start-coding)</source>
887906
<note>This is a Markdown formatted string, and the formatting must persist across translations.
888907
The fifth line should be '[TRANSLATION HERE](command:powerplatform-walkthrough.advancedCapabilities-learn-more) TRANSLATION', keeping brackets and the text in the parentheses unmodified
@@ -892,4 +911,4 @@ The seventh line should be '[TRANSLATION HERE](command:powerplatform-walkthrough
892911
<source xml:lang="en">Which Azure Cloud Power Platform Tools should use for authentication.</source>
893912
</trans-unit>
894913
</body></file>
895-
</xliff>
914+
</xliff>

src/client/power-pages/actions-hub/ActionsHub.ts

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@ import { ActionsHubTreeDataProvider } from "./ActionsHubTreeDataProvider";
1010
import { oneDSLoggerWrapper } from "../../../common/OneDSLoggerTelemetry/oneDSLoggerWrapper";
1111
import { PacTerminal } from "../../lib/PacTerminal";
1212
import { Constants } from "./Constants";
13+
import { getBaseEventInfo } from "./TelemetryHelper";
1314

1415
export class ActionsHub {
1516
private static _isInitialized = false;
@@ -32,8 +33,9 @@ export class ActionsHub {
3233
try {
3334
const isActionsHubEnabled = ActionsHub.isEnabled();
3435

35-
oneDSLoggerWrapper.getLogger().traceInfo("EnableActionsHub", {
36-
isEnabled: isActionsHubEnabled.toString()
36+
oneDSLoggerWrapper.getLogger().traceInfo(Constants.EventNames.ACTIONS_HUB_ENABLED, {
37+
isEnabled: isActionsHubEnabled.toString(),
38+
...getBaseEventInfo()
3739
});
3840

3941
vscode.commands.executeCommand("setContext", "microsoft.powerplatform.pages.actionsHubEnabled", isActionsHubEnabled);
@@ -44,10 +46,10 @@ export class ActionsHub {
4446

4547
ActionsHubTreeDataProvider.initialize(context, pacTerminal);
4648
ActionsHub._isInitialized = true;
47-
oneDSLoggerWrapper.getLogger().traceInfo(Constants.EventNames.ACTIONS_HUB_INITIALIZED);
49+
oneDSLoggerWrapper.getLogger().traceInfo(Constants.EventNames.ACTIONS_HUB_INITIALIZED, getBaseEventInfo());
4850
} catch (exception) {
4951
const exceptionError = exception as Error;
50-
oneDSLoggerWrapper.getLogger().traceError(Constants.EventNames.ACTIONS_HUB_INITIALIZATION_FAILED, exceptionError.message, exceptionError);
52+
oneDSLoggerWrapper.getLogger().traceError(Constants.EventNames.ACTIONS_HUB_INITIALIZATION_FAILED, exceptionError.message, exceptionError, getBaseEventInfo());
5153
}
5254
}
5355
}

0 commit comments

Comments
 (0)