Skip to content

#24653 - Adds a new user configuration extensions.ignoreRecommendations - which allows user to enable or disable extension recommendations#25038

Merged
joaomoreno merged 6 commits intomicrosoft:masterfrom
RishabhKothaari:master
May 4, 2017
Merged

Conversation

@RishabhKothaari
Copy link
Contributor

Adds a new USER space configuration extensions.ignoreRecommendations.
When the user clicks on Do not show again we show a message Do you want to ignore all recommendations with options Yes, Ignore All ,No and Cancel

  1. When the user chooses Yes,Ignore All - we add the current extension to extensionAssistant/importantRecommendationsIgnore list and set extensions.ignoreRecommendations to true.

  2. When the user chooses No - we add the current extension to extensionAssistant/importantRecommendationsIgnore and hide that message.

  3. When the user chooses Cancel - we just hide the current message.

In all the cases we record recommendations into extensionsAssistant/recommendations but not alert the user.

Even workspace related extension recommendations respect the value of extensions.ignoreRecommndations .

If in case the user does not choose an option for an extension and selects different file type from workspace explorer - which causes stacking up of messages, then selecting Do not show again and selecting Yes, Ignore All should not show any recommendations and hide stacked up recommendation messages.

@mention-bot
Copy link

@RISHII7, thanks for your PR! By analyzing the history of the files in this pull request, we identified @joaomoreno and @sandy081 to be potential reviewers.

@joaomoreno joaomoreno added this to the May 2017 milestone Apr 20, 2017
@joaomoreno joaomoreno self-requested a review April 20, 2017 12:04
@joaomoreno joaomoreno added the extensions Issues concerning extensions label Apr 20, 2017
Copy link
Member

@joaomoreno joaomoreno left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work! 🍻

@joaomoreno joaomoreno merged commit 052cdc7 into microsoft:master May 4, 2017
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

extensions Issues concerning extensions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants