-
Notifications
You must be signed in to change notification settings - Fork 38
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
Users/amitjoshi/GitHub copilot string updates #997
Merged
amitjoshi438
merged 24 commits into
main
from
users/amitjoshi/githubCopilotStringUpdates
Aug 9, 2024
Merged
Users/amitjoshi/GitHub copilot string updates #997
amitjoshi438
merged 24 commits into
main
from
users/amitjoshi/githubCopilotStringUpdates
Aug 9, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Moving auth create flow to utils (#926) Co-authored-by: amitjoshi <[email protected]> * Making auth provider a common module flow (#928) * refactor: Update QuickPickProvider to handle tab input changes (#930) The code changes in the QuickPickProvider class handle tab input changes by checking if the active tab is an instance of vscode.TabInputCustom or vscode.TabInputText. If it is, the QuickPickProvider updates the quick pick items based on the file path of the tab input. Note: This commit message follows the convention used in recent repository commits. * Updates to Artemis service & telemetry endpoints for gov clouds (#906) * Updates to artemis for gov clouds * change * changes --------- Co-authored-by: Shivika Gupta <[email protected]> --------- Co-authored-by: amitjoshi438 <[email protected]> Co-authored-by: amitjoshi <[email protected]> Co-authored-by: Ritik Ramuka <[email protected]> Co-authored-by: gshivi <[email protected]> Co-authored-by: Shivika Gupta <[email protected]>
* chore: Update package.json with extension dependencies and enabled API proposals for chat participants * feat: Add PowerPagesChatParticipant for chat functionality * feat: Add logic to handle chat requests in PowerPagesChatParticipant * feat: Removed logs * TODO * formatting change * Remove unused extension dependency * feat: Add handler for chat requests in PowerPagesChatParticipant --------- Co-authored-by: amitjoshi <[email protected]> Co-authored-by: tyaginidhi <[email protected]> Co-authored-by: Nidhi Tyagi 🌟🐇🌴❄️ <[email protected]>
* chore: Update package.json with extension dependencies and enabled API proposals for chat participants * feat: Add PowerPagesChatParticipant for chat functionality * feat: Add logic to handle chat requests in PowerPagesChatParticipant * feat: Removed logs * TODO * feat: Add logic to handle chat requests in PowerPagesChatParticipant(correct response format) * chore: Remove unused code and update PowerPagesChatParticipant initialization * chore: Update PowerPagesChatParticipant initialization and pac integration * feat: Initialize organization details in PowerPagesChatParticipant * chore: Remove unnecessary code and update OrgChangeNotifier initialization * Fix lint warnings * feat: Refactor PowerPagesChatParticipant class The code changes refactor the PowerPagesChatParticipant class in the `PowerPagesChatParticipant.ts` file. The changes include: - Fixing a typo in the `instance` property declaration - Updating the constructor parameters to have consistent spacing - Adding a comment to handle chat requests - Removing a console.log statement - Updating the `intializeOrgDetails` method to have consistent spacing - Updating the `intializeOrgDetails` method to use destructuring assignment - Updating the `intializeOrgDetails` method to update the `orgDetails` in the global state - Removing unused code Co-authored-by: amitjoshi <[email protected]> * Added response for error scenarios * Added & removed comments * refactor: Update PowerPagesCopilot name in package.json The code changes in the package.json file update the "name" property for the "powerpages" module to "Power Pages Copilot". This change reflects the updated name for the module. Co-authored-by: amitjoshi <[email protected]> * Code refactoring for utils and constants * Enhanced pac auth handling * refactor: Update PowerPagesChatParticipantConstants The code changes in the `PowerPagesChatParticipantConstants.ts` file refactor the constants used in the PowerPages chat participant. The changes include: - Cleaning up the code formatting Co-authored-by: amitjoshi <[email protected]> --------- Co-authored-by: amitjoshi <[email protected]> Co-authored-by: tyaginidhi <[email protected]> Co-authored-by: Nidhi Tyagi 🌟🐇🌴❄️ <[email protected]>
* chore: Update package.json with extension dependencies and enabled API proposals for chat participants * feat: Add PowerPagesChatParticipant for chat functionality * feat: Add logic to handle chat requests in PowerPagesChatParticipant * feat: Removed logs * TODO * feat: Add logic to handle chat requests in PowerPagesChatParticipant(correct response format) * feat: Add getActiveEditorContent function for retrieving active editor data The code changes add a new function called getActiveEditorContent to the Utils module. This function retrieves the content and parameters of the active editor in Visual Studio Code. It is used in the PowerPagesCopilot module to get the active file content and its associated parameters. Co-authored-by: amitjoshi <[email protected]> * chore: Remove unused code and update PowerPagesChatParticipant initialization * chore: Update PowerPagesChatParticipant initialization and pac integration * feat: Initialize organization details in PowerPagesChatParticipant * @powerpages aib endpoint handling * feat: Add getActiveEditorContent function to retrieve active editor data The code changes introduce a new function called getActiveEditorContent in the Utils module. This function allows retrieving the content and parameters of the active editor in Visual Studio Code. It is utilized in the PowerPagesCopilot module to obtain the active file content and its associated parameters. Co-authored-by: amitjoshi <[email protected]> * Multiline response handling * Todo comment for auth create exp * feat: Handle Copilot Unavailable scenario with response to user * Update in package.json * chore: Remove extension dependency * Full name for @powerpages * refactor: Refactor PowerPagesChatParticipant class - Removing unused code * chat api insider version * refactor: Refactor PowerPagesChatParticipant import paths * Update package.json version to 1.0.1-dev and other refactoring * refactor: Remove unused code in PowerPagesChatParticipant class * refactor: Moved Dataverse details fetch to utils * refactor: Added entity check to utils * refactor: Update localization strings and package version * chore: Remove @types/vscode dependency and update package-lock.json and package.json * chore: Update @types/vscode dependency to version 1.89.0 * chore: Remove @types/vscode dependency and update package-lock.json and package.json --------- Co-authored-by: amitjoshi <[email protected]> Co-authored-by: tyaginidhi <[email protected]>
* feat: Add environmentID field to OrgDetails interface The code changes add an optional `environmentID` field to the `OrgDetails` interface in the `PowerPagesChatParticipantTypes.ts` file. This field represents the ID of the environment associated with the organization. It is used in the `getEndpoint` function in the `PowerPagesChatParticipantUtils.ts` file to retrieve the intelligence endpoint and geo name for the specified organization and environment. Co-authored-by: amitjoshi <[email protected]> Co-authored-by: tyaginidhi <[email protected]> * feat: Update PowerPagesChatParticipantUtils to return componentInfo and entityName The code changes in PowerPagesChatParticipantUtils modify the getComponentInfo function to return an object with two properties: componentInfo and entityName. This change allows for better handling of component information and entity details for the active file. The updated function is used in PowerPagesChatParticipant to retrieve the necessary data for sending API requests. Co-authored-by: amitjoshi <[email protected]> * feat: Modify getComponentInfo to return componentInfo and entityName The code changes in PowerPagesChatParticipantUtils modify the getComponentInfo function to return an object with two properties: componentInfo and entityName. This change improves the handling of component information and entity details for the active file. The updated function is used in PowerPagesChatParticipant to retrieve the necessary data for sending API requests. --------- Co-authored-by: amitjoshi <[email protected]>
* feat: Add telemetry event names for GitHub Power Pages agent * chore: Update telemetry event names for GitHub Power Pages agent --------- Co-authored-by: amitjoshi <[email protected]> Co-authored-by: tyaginidhi <[email protected]>
…PagesCopilot.ts and copilot.js
tyaginidhi
requested changes
Jul 15, 2024
src/common/copilot/welcome-notification/CopilotNotificationPanel.ts
Outdated
Show resolved
Hide resolved
src/common/copilot/welcome-notification/CopilotNotificationPanel.ts
Outdated
Show resolved
Hide resolved
tyaginidhi
approved these changes
Aug 9, 2024
ritikramuka
approved these changes
Aug 9, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request primarily focuses on enhancing the user experience and localization of the Power Pages feature in GitHub Copilot. The changes include updates to prompts and messages, the addition of new translations, and codebase improvements.
User experience enhancements:
l10n/bundle.l10n.json
: Added new messages and updated existing ones to improve user interaction with GitHub Copilot.loc/translations-export/vscode-powerplatform.xlf
: Updated translations for various prompts and messages. [1] [2] [3] [4] [5] [6] [7] [8]package.json
: Updated the full name and description of the Power Pages Copilot feature.src/common/chat-participants/powerpages/PowerPagesChatParticipant.ts
: Implemented a follow-up provider for the chat participant and updated the initial prompt for users. [1] [2] [3]src/common/chat-participants/powerpages/PowerPagesChatParticipantConstants.ts
: Updated the initial prompt and added new constants for various messages.Codebase improvements:
src/common/chat-participants/powerpages/PowerPagesChatParticipant.ts
: Removed unnecessary code and improved code readability. [1] [2]src/common/chat-participants/powerpages/PowerPagesChatParticipantUtils.ts
: Updated imports and added new functionality for handling starter prompts.