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

x/tour: welcome/3 slide missing instructions for installing Go #34585

Closed
dmitshur opened this issue Sep 28, 2019 · 1 comment
Closed

x/tour: welcome/3 slide missing instructions for installing Go #34585

dmitshur opened this issue Sep 28, 2019 · 1 comment
Labels
Documentation FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Milestone

Comments

@dmitshur
Copy link
Contributor

The first slide of the tour welcomes people and explains how to change slides. The second slide mentions the choice of spoken languages. The third slide strongly suggests running the tour offline.

By the third slide, the tour assumes users already have Go installed (and/or know how to install Go). This isn't valid for users who are being introduced to Go by being linked to the tour. It can cause unnecessary confusion as demonstrated by the experience reported in this comment.

That slide used to have instructions for installing Go, but they were lost as part of deployment changes in CL 141857. /cc @andybons

To fix this, we should include a link to https://golang.org/doc/install and/or https://golang.org/dl/ before the first mention of go get in the tour. It should also be helpful and a good teaching opportunity to point out where the tour binary ends up after go get runs successfully.

This issue is factored out from #24819, which was about a formatting bug that is resolved by now.

@dmitshur dmitshur added Documentation NeedsFix The path to resolution is known, but the work has not been done. labels Sep 28, 2019
@dmitshur dmitshur added this to the Unreleased milestone Sep 28, 2019
@dmitshur dmitshur self-assigned this Sep 28, 2019
@gopherbot
Copy link

Change https://golang.org/cl/197857 mentions this issue: content: make welcome/3 slide more beginner friendly

@golang golang deleted a comment Sep 30, 2019
@golang golang locked and limited conversation to collaborators Sep 30, 2020
gopherbot pushed a commit to golang/website that referenced this issue May 26, 2021
Re-add the link to installing Go on the welcome/3 slide,
because it's a pre-requisite for being able to run go get.

This was included before, but got lost in a refactor after
the tour binary moved out of the Go distribution.

Additionally, make it more clear that installing the tour
locally is an optional step that only needs to be done if
one wishes to run the tour offline; it's not mandatory to
get through the tour. Running the tour online has a lower
overhead, and should help keep the tour accessible to more
people.

Fixes golang/go#34585
Updates golang/go#24819

Change-Id: If5f51e9e876722a86afca054bf78428f78bb3076
Reviewed-on: https://go-review.googlesource.com/c/tour/+/197857
Reviewed-by: Emmanuel Odeke <emm.odeke@gmail.com>
X-Tour-Commit: 45121acd746573507cdaaf902958dba082f7c25b
passionSeven added a commit to passionSeven/website that referenced this issue Oct 18, 2022
Re-add the link to installing Go on the welcome/3 slide,
because it's a pre-requisite for being able to run go get.

This was included before, but got lost in a refactor after
the tour binary moved out of the Go distribution.

Additionally, make it more clear that installing the tour
locally is an optional step that only needs to be done if
one wishes to run the tour offline; it's not mandatory to
get through the tour. Running the tour online has a lower
overhead, and should help keep the tour accessible to more
people.

Fixes golang/go#34585
Updates golang/go#24819

Change-Id: If5f51e9e876722a86afca054bf78428f78bb3076
Reviewed-on: https://go-review.googlesource.com/c/tour/+/197857
Reviewed-by: Emmanuel Odeke <emm.odeke@gmail.com>
X-Tour-Commit: 45121acd746573507cdaaf902958dba082f7c25b
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Documentation FrozenDueToAge NeedsFix The path to resolution is known, but the work has not been done.
Projects
None yet
Development

No branches or pull requests

2 participants