Skip to content

Show dialog to select customizations - #33

Merged
breedloj merged 11 commits into
mainfrom
paras/showCustomizationsDialog
Sep 30, 2024
Merged

Show dialog to select customizations#33
breedloj merged 11 commits into
mainfrom
paras/showCustomizationsDialog

Conversation

@pras0131

@pras0131 pras0131 commented Sep 24, 2024

Copy link
Copy Markdown
Contributor

Reference ticket: https://issues.amazon.com/issues/ECLIPSE-294

  • The PR aims to show the Select Customization option in the menu bar, and clicking on the same opens up a dialog to select customizations.
  • Right now, the implementation of Pro Login via IAM Identity control center does not exist, so the visibility of the customization menu option & the corresponding dialog has been put behind just isLogin condition (which checks the login via BuilderID)
  • A little about functionality and some edge cases in the PR: The dropdown will be disabled if Amazon Q Foundation (default) is selected. Option to select the customisation (from dropdown) is only enabled when customization radio button is selected. The existing DialogContributionItem has not been used, because we need to do some pre-action before opening up the dialog like making a call to the LSP to fetch the list of customizations and setting the initial state of radio button (or pre-chosen customization in case of customization radio button) auto selection based on the saved preferences.
  • Selected customization will be saved in the Preference Store. Preference store customization configuration will be deleted if the user selects the default configuration, which is Amazon Q Foundation.
  • Right now, there is no call to LSP to fetch the configurations. Currently the response is mocked..Call to LSP to fetch the customizations will be done in a separate PR. Also, right now pressing Select option saves the preference in a preference store but notification to LSP on change of configuration will be done separately in a different PR.
  • Attaching the screenshots below for reference:
Screenshot 2024-09-24 at 16 08 14 Screenshot 2024-09-30 at 15 02 32

@pras0131
pras0131 requested a review from breedloj September 24, 2024 14:24
@breedloj

Copy link
Copy Markdown
Contributor

Is the screenshot up to date and is the dialog now reflecting Jo's latest comments?

@pras0131

Copy link
Copy Markdown
Contributor Author

Is the screenshot up to date and is the dialog now reflecting Jo's latest comments?

Updated the screenshot and fixed the UI based on Jo's latest comments.

@breedloj
breedloj merged commit fb4ba6c into main Sep 30, 2024
@breedloj
breedloj deleted the paras/showCustomizationsDialog branch September 30, 2024 16:56
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.

2 participants