-
Notifications
You must be signed in to change notification settings - Fork 176
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
feat: migrate cli fetch #748
Conversation
loks0n
commented
Dec 27, 2023
- Migrate CLI from axios to fetch
- Refactor commands.js.twig to match patterns from Node and Web templates
@loks0n, would you be able to look into why the node tests are failing? |
…o feat-migrate-cli-fetch
@@ -163,6 +163,7 @@ const commandDescriptions = { | |||
"locale": `The locale command allows you to customize your app based on your users' location.`, | |||
"projects": `The projects command allows you to view, create and manage your {{ spec.title|caseUcfirst }} projects.`, | |||
"storage": `The storage command allows you to manage your project files.`, | |||
"messaging": `The messaging command allows you to manage and send messages to your users`, |
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.
we have to remove 1.5 related items from here. The fixed CLI needs a release before 1.5
@@ -6,7 +6,7 @@ const { globalConfig, localConfig } = require("../config"); | |||
const { actionRunner, success, parseBool, commandDescriptions, log, parse } = require("../parser"); | |||
{% if sdk.test != "true" %} | |||
const { questionsLogin } = require("../questions"); | |||
const { accountCreateEmailSession, accountDeleteSession } = require("./account"); | |||
const { accountCreateEmailPasswordSession, accountDeleteSession } = require("./account"); |
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.
Is this for 1.5 ? If not lets remove it and change the base branch to main
This PR has been refactored to branch |