Skip to content

Changing README file #410

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.

Sign up for GitHub

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

Jump to bottom
Merged
uhrjun merged 1 commit into from
Apr 19, 2023
Merged

Changing README file #410

uhrjun merged 1 commit into from
Apr 19, 2023

Conversation

@Sorizz
Copy link
Contributor

@Sorizz Sorizz commented Apr 17, 2023

Explanation About the Issue
  • I have tried to install frappe charts on my machine locally
  • I encountered this issue npm-run-all is not a recognizable command
  • So I added a line of code to resolve this issue

Regards,

    I hope this little contribution of mine would help newcomers to setup frappe chart locally without any heckle.

@uhrjun uhrjun merged commit 8f69be0 into frappe:master Apr 19, 2023
@uhrjun
Copy link
Contributor

uhrjun commented Apr 19, 2023

Thanks for this 👍

@Sorizz
Copy link
Contributor Author

Sorizz commented Apr 20, 2023

Thanks for being my

First PR 🔥

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@Sorizz @uhrjun