Skip to content

New auth flow#430

Open
chinmay29fynd wants to merge 17 commits into
masterfrom
new-auth-flow
Open

New auth flow#430
chinmay29fynd wants to merge 17 commits into
masterfrom
new-auth-flow

Conversation

@chinmay29fynd

Copy link
Copy Markdown
Collaborator

New auth flow

Comment thread src/lib/Auth.ts Outdated
Comment thread src/lib/Auth.ts Outdated

@vivek8690 vivek8690 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A few notes on keeping this consistent with how the rest of the CLI is structured. Main one is the URL building — see inline.

Comment thread src/lib/Auth.ts Outdated
Comment thread src/lib/Auth.ts Outdated
Comment thread src/lib/Auth.ts Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants