Improve CLI user experience & create continuity between control panel and CLI #1820
Conversation
🦋 Changeset detectedLatest commit: 7f665f9 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
3 Skipped Deployments
|
bookernath
force-pushed
the
cli-occ-apis
branch
from
a1bbc9c to
3c451f6
Compare
|
|
||
| if (this.config.auth?.storeHash && this.config.auth.accessToken) { | ||
| configObj.auth = { | ||
| storeHash: this.config.auth.storeHash, |
There was a problem hiding this comment.
Will the same store hash be in the main .env file and this config? If it is, do we need the store hash in both?
There was a problem hiding this comment.
The utility of the .catalyst file is you can regenerate the .env.local, so it makes sense for it to have no dependencies on .env.local
bookernath
force-pushed
the
cli-occ-apis
branch
from
a2b801f to
8bb29d8
Compare
|
Appreciate the changes post review. Flow looks 🧈 |
bookernath
force-pushed
the
cli-occ-apis
branch
from
8bb29d8 to
7f665f9
Compare
⚡️🏠 Lighthouse reportLighthouse ran against https://catalyst-latest-9sa5wispd-bigcommerce-platform.vercel.app 🖥️ DesktopWe ran Lighthouse against the changes on a desktop and produced this report. Here's the summary:
📱 MobileWe ran Lighthouse against the changes on a mobile and produced this report. Here's the summary:
|
What/Why?
.catalystfile with store hash/access token after login action to be used for other CLI commands without re-prompting for loginTesting
Create wizard (the extraneous log statement is not there in this actual PR)
createwizard.mov
Starting from app (I invalidated these credentials before posting this)
occ-to-cli.mov
Init command
init.mov