Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove outline from feedback button styles #1150

Merged
merged 2 commits into from
Mar 12, 2025
Merged

Conversation

priyanshu92
Copy link
Contributor

@priyanshu92 priyanshu92 commented Mar 11, 2025

  • ✨ Removed the outline property from button styles for improved accessibility.

image

Color contrast ratio of outline
image

@priyanshu92 priyanshu92 requested review from a team as code owners March 11, 2025 16:19
- ✨ Removed the outline property from button styles for improved accessibility.

-Priyanshu
@priyanshu92 priyanshu92 enabled auto-merge (squash) March 11, 2025 16:21

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@priyanshu92 priyanshu92 merged commit f733158 into main Mar 12, 2025
6 checks passed
@priyanshu92 priyanshu92 deleted the users/priyanshuag/outline branch March 12, 2025 10:26
sanxsth added a commit that referenced this pull request Mar 18, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
* 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

---------

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: Priyanshu Agrawal <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Sandeep Satheesh <[email protected]>
Co-authored-by: Priyanshu Agrawal <[email protected]>
Co-authored-by: BidishaMS <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants