Skip to content
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

[RFR] Update tutorial app creation command #4187

Closed
wants to merge 1 commit into from
Closed

[RFR] Update tutorial app creation command #4187

wants to merge 1 commit into from

Conversation

jpetitcolas
Copy link
Contributor

In the tutorial, create-react-app is installed globally. Yet, it is no longer supported as shown with the following error message:

Please note that global installs of create-react-app are no longer supported.

This PR uses npx, the way recommended by create-react-app.

In the tutorial, `create-react-app` is installed globally. Yet, it is no longer supported as shown with the following error message:

> Please note that global installs of create-react-app are no longer supported.

This PR uses `npx`, the way recommended by [create-react-app](https://github.com/facebook/create-react-app).
@jpetitcolas jpetitcolas changed the title Update tutorial app creation command [RFR] Update tutorial app creation command Dec 18, 2019
@@ -12,8 +12,7 @@ This 30 minutes tutorial will expose how to create a new admin app based on an e
React-admin uses React. We'll use Facebook's [create-react-app](https://github.com/facebookincubator/create-react-app) to create an empty React app, and install the `react-admin` package:

```sh
npm install -g create-react-app
create-react-app test-admin
npx create-react-app test-admin
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
npx create-react-app test-admin
npx create-react-app test-admin
# or
yarn create react-app test-admin
# or
npm init react-app test-admin

Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Didn't know about these extra commands. Thanks! That being said, I'm not sure we should be so exhaustive to keep it simple. But your choice! :)

Copy link
Collaborator

Choose a reason for hiding this comment

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

As a yarn user, I only knew about npx and yarn create. Just discovered npm had an init :)

@fzaninotto
Copy link
Member

Superseded by #4198

@fzaninotto fzaninotto closed this Mar 30, 2020
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.

3 participants