Skip to content

fix(docs): update zeit now deployment process considering v2 #3574

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

Closed
wants to merge 3 commits into from
Closed

fix(docs): update zeit now deployment process considering v2 #3574

wants to merge 3 commits into from

Conversation

franciscobrba
Copy link
Contributor

Now is currently on version 2, and it use that version by default when someone want to make a deploy, then it let us two choices:

  • A) Maintain current docs about v1 deployment but set version on example JSON.
  • B) Update docs considering the the advent of v2 (11/2018)

I choose option B, then i made the changes and just commit it.

Before

captura de tela 2019-03-05 as 00 24 51

After

captura de tela 2019-03-05 as 00 25 06

Reasons to try to contribute

  • Docs need to be always updated :)

Refs

ZEIT - Now about Vue.js deployment (v2)

Sorry my english, only a Brazilian servant.

@LinusBorg
Copy link
Member

Possible duplicate with #3560

@franciscobrba
Copy link
Contributor Author

Possible duplicate with #3560

To work right must exists a "now-build" script on package.json #3560 don't have this warn/instruction.

@Virgiel
Copy link

Virgiel commented Mar 17, 2019

I think you are right to use the option B as I see no reason not to get the advantage of now 2 with a vue project

@Virgiel
Copy link

Virgiel commented Mar 20, 2019

Now CLI has just reached version 14.2.0 and the aliasing commands has changed

now build and give a default alias
now --target production build and give alias from the config

more information at https://zeit.co/blog/automatic-aliasing

@NataliaTepluhina
Copy link
Member

Thank you for working on this! Unfortunately, need to close the PR in favor of #4232

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants