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/pkgsite: adding a package via request button description is misleading #60552

Open
bernhardreiter opened this issue Jun 1, 2023 · 5 comments
Labels
NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made. pkgsite

Comments

@bernhardreiter
Copy link

bernhardreiter commented Jun 1, 2023

What is the URL of the page with the issue?

https://pkg.go.dev/about#adding-a-package

What is your user agent?

Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/113.0.0.0 Safari/537.36

What did you do?

Tried to follow the first variant for adding a package, which says:

Visiting that page on pkg.go.dev, and clicking the “Request” button. For example:
https://pkg.go.dev/example.com/my/module

Went to a couple of those modules and searched for the "Request" button

Could not find the "Request" button.

What did you expect to see?

That any url that matches the example URL of the documentation works like it is written there and offers a "Request" button.

Or that the documentation tells me instead, under which conditions the button should appear.

Or the documentation is corrected to remove this possibility, if it is gone by now.

What did you see instead?

Searching "Request" on the pages did yield results.

@gopherbot gopherbot added this to the Unreleased milestone Jun 1, 2023
@bernhardreiter bernhardreiter changed the title x/pkgsite: x/pkgsite: adding a package via request button does not work Jun 1, 2023
@jamalc
Copy link

jamalc commented Jun 1, 2023

The packages you referenced are available on pkg.go.dev. Were they unavailable when you filed this issue?

@jamalc jamalc added the WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided. label Jun 1, 2023
@bernhardreiter
Copy link
Author

The packages you referenced are available on pkg.go.dev. Were they unavailable when you filed this issue?

They were available.
Ah, now I see how the text made me think in the wrong way.

Going to https://pkg.go.dev/github.com/DTGlov/go-basics there is a "Request" button.

My use case was to add a new version of a package and the instructions do not explicitly mention this case. The second and third variant in the instructions under "adding package" seem to refer to a specific version, so I guess they can be used to update to a new major version of the package. The first variant at least cannot.

My suggestion to improve the text is to add a condition when the "request" buttons is shown, e.g. make it read like:

Try the page you expect on pkg.go.dev, which will offer a “Request” button
in case the package is not listed at all. Then click it. For example:
https://pkg.go.dev/example.com/my/module

A different way to fix this would be to mention how packages are updated instead of added.

@bernhardreiter
Copy link
Author

bernhardreiter commented Jun 6, 2023

@jamalc with the last message I've given the requested information, how is the label WaitingforInfo removed now? (I lack permission to remove it.)

@jamalc
Copy link

jamalc commented Jun 6, 2023

Would adding the version string or an additional example with the version string make it clear?

Adding a package

Data for the site is downloaded from [proxy.golang.org](https://proxy.golang.org/).
We monitor the [Go Module Index](https://index.golang.org/index) regularly for new
packages to add to pkg.go.dev. If you don’t see a package on pkg.go.dev, you can
add it by doing one of the following:

Visiting that page on pkg.go.dev, and clicking the “Request” button. For example:
https://pkg.go.dev/example.com/my/module@v1.0.0

@jamalc jamalc modified the milestones: Unreleased, pkgsite/later Jun 6, 2023
@jamalc jamalc added NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made. and removed WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided. labels Jun 6, 2023
@bernhardreiter
Copy link
Author

Would adding the version string or an additional example with the version string make it clear?

It would be an improvement, because a "Request" button appears if the url is used with a version number that is not yet found.

Even better would be if there was a mention that this also works to add another - like a new - version of a package.
Maybe like

If you don’t see a specific version of a package on pkg.go.dev, you can
add it by doing one of the following:

@bernhardreiter bernhardreiter changed the title x/pkgsite: adding a package via request button does not work x/pkgsite: adding a package via request button description is misleading Jun 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NeedsDecision Feedback is required from experts, contributors, and/or the community before a change can be made. pkgsite
Projects
None yet
Development

No branches or pull requests

3 participants