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

[DCR]: Adding a new Package Source in VS Options should be immediately available for a Package Source Mapping #12155

Closed
donnie-msft opened this issue Oct 12, 2022 · 7 comments
Assignees
Labels
Area:PackageSourceMapping Issues related to the package source mapping feature Functionality:VisualStudioUI Functionality:VSOptions Priority:2 Issues for the current backlog. Product:VS.Client Type:DCR Design Change Request

Comments

@donnie-msft
Copy link
Contributor

NuGet Product(s) Affected

Visual Studio Package Management UI

Current Behavior

As described in discussion on the PR creating a new Package Source Mappings page in VS Options,
NuGet/NuGet.Client#4847 (comment), Package Sources do not commit immediately to a config file. Therefore, a newly added source cannot be immediately used in a mapping, today.

Desired Behavior

A user should be able to add a new source and immediately create a Package Source Mapping for it, without having to press OK and re-open the options.

Additional Context

No response

@donnie-msft donnie-msft added Product:VS.Client Priority:3 Issues under consideration. With enough upvotes, will be reconsidered to be added to the backlog. Type:DCR Design Change Request Functionality:VisualStudioUI Area:PackageSourceMapping Issues related to the package source mapping feature labels Oct 12, 2022
@donnie-msft donnie-msft self-assigned this Oct 12, 2022
@nkolev92 nkolev92 added Priority:2 Issues for the current backlog. and removed Priority:3 Issues under consideration. With enough upvotes, will be reconsidered to be added to the backlog. labels Dec 7, 2022
@donnie-msft
Copy link
Contributor Author

#11361

@zivkan
Copy link
Member

zivkan commented Sep 26, 2023

Is this still an issue since #1538 was implemented?

@donnie-msft
Copy link
Contributor Author

Is this still an issue since #1538 was implemented?

1538 is for PM UI and 12155 is for the Options dialog. What do you think?

@zivkan
Copy link
Member

zivkan commented Sep 27, 2023

This issue says that when some setting is changed, customers have to close and reopen the solution. The implementation for #1538 uses a file watcher on the singleton settings, so regardless of what caused the nuget.config file to change, VS should immediately reload the files and therefore any changes should apply to any NuGet VS component that uses that singleton settings instance. Therefore, I would expect that the behaviour described in this issue no longer occurs.

@donnie-msft
Copy link
Contributor Author

This issue says that when some setting is changed, customers have to close and reopen the solution.

Not the solution, just the Options dialog.

@donnie-msft
Copy link
Contributor Author

I just tested it to make sure I'm not missing something: Options still doesn't pick up a newly added package source. Just to reiterate: This is about the Options dialog, not the PM UI, and the file watcher won't affect the behavior there. Options loads sources from disk once, then writes them after pressing "OK" on the dialog.

@donnie-msft
Copy link
Contributor Author

Closing for now as this isn't planned and somehow keeps reappearing on my board.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area:PackageSourceMapping Issues related to the package source mapping feature Functionality:VisualStudioUI Functionality:VSOptions Priority:2 Issues for the current backlog. Product:VS.Client Type:DCR Design Change Request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants