This repository has been archived by the owner on Jun 21, 2023. It is now read-only.
Show proper message when repo is created #59
Closed
Closed
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
Adds a list view showing the list of repositories that VS knows about and that are associated with the github connection (i.e., the remote url corresponds to the github connection section information)
Conflicts: src/GitHub.Exports/Primitives/UriString.cs src/GitHub.Exports/Services/Services.cs src/GitHub.VisualStudio/Base/TeamExplorerItemBase.cs
The repository list is loaded from the registry into ConnectionManager. Don't save it in the cache for now, so it doesn't break compatibility with the existing cache files and we don't really need to cache it right now. Let's see how it works just loading directly from the registry for now.
Offering to open a solution or project is the only way to switch the context to the new repo. Since it's likely that that's what the user wants to do anyway (why clone a repo otherwise), let's do that.
The previous logic wouldn't work if there is no active repo in VS
Also fix some issues with refreshing the repo list multiple times and tracking when local repository list for changes
Make only the repo name bold when active. Change the repo path color to white when hovering or selecting so it's more visible. Set the background to selected when selecting with the mouse (different from just hovering).
Keep the path color white/black when it's the current repo, it's easier to see.
if (isCreating) | ||
{ | ||
vsservices.ClearNotifications(); | ||
vsservices.ShowMessage(string.Format(CultureInfo.CurrentUICulture, "[{0}]({1} has been successfully created.", SelectedRepository.Name, SelectedRepository.CloneUrl)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the only piece of code in this PR, everything else comes from #32
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In the future, you can target one PR towards another PR's branch. That way only this one commit would show up. 😄
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Fixes #54
This PR relies on shana/list-git-repos