napi login command
#147
napi login command
#147
Conversation
| import { input } from "npm:@inquirer/prompts"; | ||
|
|
||
| function isValidEmail(email: string): boolean { | ||
| return /^[^\s@]+@[^\s@]+\.[^\s@]+$/.test(email); |
There was a problem hiding this comment.
We can use zod for this, we already use it in the CLI for other stuff
| }, | ||
| }); | ||
|
|
||
| const response = await fetch("https://api.nanoapi.io/v1/auth/requestOtp", { |
There was a problem hiding this comment.
Need some global option to override this in the CLI, usefull for working locally, will also be used for poeple having self hosted version in the future
| return; | ||
| } | ||
|
|
||
| const resposne = await fetch(url, { |
| } | ||
| } | ||
|
|
||
| export function setUserAuthToken(token: string) { |
There was a problem hiding this comment.
Here we could also setup the user email from the token we get.
Usefull for when we send telemetry
packages/cli/src/cli/index.ts
Outdated
| host: { | ||
| type: "string" as PositionalOptionsType, | ||
| default: "https://api.nanoapi.io", | ||
| alias: "h", |
There was a problem hiding this comment.
This alias already taken by the defaul "help" command
Maybe we can do "api_host" and helper "ah"?
Or no alias at all
There was a problem hiding this comment.
for info, the help command is not codded, handled by yargs
There was a problem hiding this comment.
Ah alright good to know
There was a problem hiding this comment.
I've set it to uppercase -H. This should be fine for now and if we need to change it in a future release we can rename it to something else then.
Type of change
not work as expected)
Description
Please include a summary of the changes and the related issue. Explain the
motivation behind this change.
Related Issue
Issue Number: #<issue_number>
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Checklist