-
Notifications
You must be signed in to change notification settings - Fork 3.4k
docs: clarify legacy token creation in npm login and adduser commands #8226
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
docs: clarify legacy token creation in npm login and adduser commands #8226
Conversation
@@ -14,6 +14,8 @@ Create a new user in the specified registry, and save the credentials to | |||
the `.npmrc` file. If no registry is specified, the default registry | |||
will be used (see [`registry`](/using-npm/registry)). | |||
|
|||
When you run `npm adduser`, the CLI automatically generates a legacy token of publish type. For more information, see [About legacy tokens](/about-access-tokens#about-legacy-tokens). |
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.
When you run `npm adduser`, the CLI automatically generates a legacy token of publish type. For more information, see [About legacy tokens](/about-access-tokens#about-legacy-tokens). | |
When you run `npm adduser`, the CLI automatically generates a legacy token of `publish` type. For more information, see [About legacy tokens](/about-access-tokens#about-legacy-tokens). |
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.
Done
@@ -14,6 +14,8 @@ Verify a user in the specified registry, and save the credentials to the | |||
`.npmrc` file. If no registry is specified, the default registry will be | |||
used (see [`config`](/using-npm/config)). | |||
|
|||
When you run `npm login`, the CLI automatically generates a legacy token of publish type. For more information, see [About legacy tokens](/about-access-tokens#about-legacy-tokens). |
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.
When you run `npm login`, the CLI automatically generates a legacy token of publish type. For more information, see [About legacy tokens](/about-access-tokens#about-legacy-tokens). | |
When you run `npm login`, the CLI automatically generates a legacy token of `publish` type. For more information, see [About legacy tokens](/about-access-tokens#about-legacy-tokens). |
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.
Done
This is simple PR for adjust
npm login
andnpm adduser
doc.Problem Description:
When using
npm login
/npm adduser
, users are not aware that this command generates a legacy token with publish permissions. This information is not evident in thenpm login
/npm adduser
cli documentation.After a while searching. found that this context and detail hiding in the About legacy tokens documentation.
Solution:
Add a clear description in the
npm login
/npm adduser
documentation in this PR so that users can understand publish token would be created afternpm login
in the future.References
Related to this PR: npm/documentation#1550