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

Suggestion: Add note about creating SSH keys to the "Clone a repo" onboarding task #151453

Open
hawkticehurst opened this issue Jun 7, 2022 · 8 comments
Assignees
Labels
feature-request Request for new features or functionality git GIT issues
Milestone

Comments

@hawkticehurst
Copy link
Member

hawkticehurst commented Jun 7, 2022

When completing the clone a repo task in the onboarding experience, the item was marked as complete once I pasted in a GitHub URL and clicked enter, but I ended up running into a (somewhat cryptic) error immediately afterward because I didn't have my GitHub SSH keys set up on my machine yet.

I figure this will be a non-issue for most developers who have had the experience of setting up a new dev machine more than a few times. However, I could see that being an incredibly confusing state to end up in if I were a new coder just getting started.

As a result, I think it might be really nice to explore the inclusion of some kind of note telling people to make sure they have SSH keys correctly configured (with links to the relevant GitHub docs) to hopefully keep the VS Code onboarding process smooth for new devs.

@hawkticehurst hawkticehurst changed the title Suggestion: Add note about creating SSH keys when completing "Clone a repo" onboarding task Suggestion: Add note about creating SSH keys to the "Clone a repo" onboarding task Jun 7, 2022
@meganrogge meganrogge assigned tanhakabir and unassigned meganrogge Jun 7, 2022
@tanhakabir tanhakabir assigned joyceerhl and unassigned tanhakabir Jun 7, 2022
@joyceerhl joyceerhl assigned lszomoru and unassigned joyceerhl Jun 7, 2022
@daviddossett
Copy link
Contributor

cc @digitarald

This may be two-pronged:

  • Should the signal for completion come later than entering the URL?
  • We may need to tune the error to be actionable/readable? @hawkticehurst got a screenshot of the error?

@hawkticehurst
Copy link
Member Author

hawkticehurst commented Jun 7, 2022

No, I unfortunately don't but let me see if I can recreate it

Also to clarify, this was a Windows OS error dialog that got rendered -- not VS Code. Thus the suggestion to preemptively add a note about setting up SSH keys since you won't have control over the error message.

@daviddossett
Copy link
Contributor

It's possible we're still the ones firing the notification though. Or maybe it was super clear that it came from the OS?

In any case, the reason I say that is because notifications can get rendered as modals e.g. vscode.window.showInformationMessage('Info message', { modal: true });

CleanShot 2022-06-07 at 15 12 35@2x

@hawkticehurst
Copy link
Member Author

hawkticehurst commented Jun 7, 2022

Oh touché! It did look very Windows OS to me, however, I'm super unfamiliar with Windows so it definitely could still have been fired from VS Code now that I think about it

@hawkticehurst
Copy link
Member Author

hawkticehurst commented Jun 7, 2022

Nice, you were totally right @daviddossett, it is a dialog triggered from VS Code

I recreated the issue by uninstalling git and VS Code and retracing my steps from scratch, and I have a few updates:

First, it looks like the "Track your code with Git" task is actually marked as completed once you click "Clone repository" button -- not when you enter the URL.

vscode-clone-repo

Second, as I said above this does actually seem to be a VS Code triggered dialog

vscode-clone-repo-error

Third, upon doing some more googling I think this is an ever so slightly different issue than "SSH keys not being configured" and more specifically is about github.com not being a part of the ssh known hosts file -- read more about it here

I believe creating SSH keys still solves the problem, but just wanted to be more precise in what the exact issue is

(FYI, the image below shows the result of clicking both the "Open Git Log" and "Show Command Output" buttons from the last image)

vscode-clone-repo-log-and-command-info

@hawkticehurst
Copy link
Member Author

With that new info, I agree with David.

I think tuning the error to be actionable/readable/more beginner-friendly could a long way in improving the onboarding process since I kind of suspect newer coders are the ones that will most likely be running into the error anyway

@digitarald
Copy link
Contributor

@lszomoru could we catch that auth error and guide users to the proper configuration?

@lszomoru
Copy link
Member

@digitarald, I have to look into it, but I think that we might be able to catch this error condition (in a similar fashion as we catch the error condition when you try to commit but do not have name/email configured) and be more helpful to the user.

@lszomoru lszomoru added feature-request Request for new features or functionality git GIT issues labels Jun 13, 2022
@lszomoru lszomoru added this to the Backlog milestone Jun 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Request for new features or functionality git GIT issues
Projects
None yet
Development

No branches or pull requests

8 participants