Skip to content
This repository was archived by the owner on Jul 28, 2021. It is now read-only.

Conversation

@alopezsanchez
Copy link
Contributor

@alopezsanchez alopezsanchez commented Nov 13, 2018

Ref: https://www.npmjs.com/package/npm-profile
Ref: https://docs.npmjs.com/cli/profile.html
Ref: https://docs.npmjs.com/cli/token.html

  • adduser
  • login
  • adduserWeb
  • loginWeb
  • adduserCouch
  • loginCouch
  • get
  • set
    • enable-2fa
    • disable-2fa
    • set password
  • listTokens
  • removeToken
  • createToken

@alopezsanchez
Copy link
Contributor Author

Hello @zkat, I have some questions.

  • I'm not sure how to implement login* and adduser* subcommands. I don't understand the opener and prompter arguments.
  • Should we add the three types of login/adduser?

Thanks!

@alopezsanchez alopezsanchez changed the title WIP: add profile command feat(profile): add profile subcommands Nov 19, 2018
@alopezsanchez
Copy link
Contributor Author

@zkat If you think that login and adduser subcommands are out of scope, I think you can review this! 😃

Btw, I have some improvements in my TODO list:

  • Tests
  • Improve CLI UX with ink
  • Improve tink profile get table
  • Logging


async function enable2fa (argv) {
const url = require('url')
const queryString = require('query-string')
Copy link
Contributor

Choose a reason for hiding this comment

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

Why not use the built-in querystring module?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You are right. Changed 👍


const getOptions = argv => ProfileConfig(npmConfig().concat(argv).concat({ log }))

// TODO: Move to `../utils`?
Copy link
Contributor

Choose a reason for hiding this comment

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

Nah

Copy link
Contributor Author

Choose a reason for hiding this comment

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

OK!

Copy link
Contributor

@zkat zkat left a comment

Choose a reason for hiding this comment

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

So far so good!

@zkat zkat merged commit a33cca1 into npm:latest Nov 20, 2018
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.

2 participants