Skip to content
This repository was archived by the owner on Jun 21, 2023. It is now read-only.

Conversation

@StanleyGoldman
Copy link
Contributor

@StanleyGoldman StanleyGoldman commented May 22, 2019

Add Publish to GitHub section to Team Explorer - Synchronization page:

image

When Publish to GitHub button is clicked:

image

When user opens the Team Explorer - Synchronization page, only GitHub.VisualStudio.16 will be loaded:

image

How to test

  1. Ensure that only the GitHub Essentials extension is installed
  2. Create a new project
  3. Add to Source Control > Git
  4. Click Publish to GitHub button
    image
  5. Choose a server (GitHub or GitHub Enterprise), owner (user or organization) and repository name
  6. Check Private (if you don't want your test repository to be publicly visible)
  7. Click Publish button
    image
  8. Expect Repository created successfully message
    image
  9. Click on Sync button
  10. Expect no publish sections to be visible
    image

Copy link
Contributor Author

@StanleyGoldman StanleyGoldman left a comment

Choose a reason for hiding this comment

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

Something I noticed, do we need a ParentSection

@meaghanlewis
Copy link
Contributor

I tested this out and it's mainly working as expected. Two things I experienced.

First, the GitHub icon isn't present on the Sync page.
Screen Shot 2019-06-10 at 10 42 28 AM

Second, the Add/Change Accounts option does allow me to sign in to a different account but after I signed in with my enterprise account and went back to select where to publish I didn't see the enterprise account as an option.
Screen Shot 2019-06-10 at 10 43 17 AM

@jcansdale
Copy link
Collaborator

jcansdale commented Jun 24, 2019

We seem to have an issue with the Blue theme.

Dark looks fine:

image

Light looks fine:

image

Blue (the default) looks washed out and grey:

image

I was hoping to avoid special casing the color themes by using the build in CrispImage component.

The image used on Browse a repository looks fine:

image

We should be using the same image! 😕

Couldn't get CrispImage to work for automatic coloring of the Blue theme.
When a user Add/Changes Account, automatically change the host drop-down.
@jcansdale
Copy link
Collaborator

jcansdale commented Jun 25, 2019

@meaghanlewis,

First, the GitHub icon isn't present on the Sync page.

This is surprising. I wonder if you somehow ended up with two versions installed (this can happen after the extension name change). How did you install this version for testing?

Second, the Add/Change Accounts option does allow me to sign in to a different account but after I signed in with my enterprise account and went back to select where to publish I didn't see the enterprise account as an option.

I've fixed it so that after you Add/Change Accounts, it will change the drop down list of hosts to the connection you just logged in using. Previously the new connection should have appeared in the list, but you would have had to select it manually.

I hope the unexpected behavior was it not being automatically selected, rather than it not appearing in the list at all.

image

@meaghanlewis
Copy link
Contributor

@jcansdale this looks good to me! I'm not experiencing the problems I was previously. Perhaps I didn't notice the account picker before.

One small thing that I noticed was that for GitHub repositories the repo name was autofilled, but for GHE it wasn't.

Only load GitHub.Exports when composition container is created.
This avoids loading the GitHub.Extensions and GitHub.Exports assemblies.
@jcansdale
Copy link
Collaborator

One small thing that I noticed was that for GitHub repositories the repo name was autofilled, but for GHE it wasn't.

I'm guessing the issue is here, but I don't understand exactly what's going wrong:
https://github.com/github/VisualStudio/blame/master/src/GitHub.App/ViewModels/TeamExplorer/RepositoryPublishViewModel.cs#L96

This will have been a long standing issue.

@jcansdale jcansdale marked this pull request as ready for review September 13, 2019 11:47
@jcansdale jcansdale changed the title Adding publish to essentials plugin Adding Publish to Essentials extension Sep 26, 2019
Copy link
Collaborator

@jcansdale jcansdale left a comment

Choose a reason for hiding this comment

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

I've check and double checked. It seems to be working well. LGTM!

@jcansdale jcansdale merged commit 126a894 into master Sep 27, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants