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

add steps to npm install and build to CONTRIBUTING.md #19

Closed
wants to merge 1 commit into from

Conversation

batpad
Copy link
Collaborator

@batpad batpad commented Mar 28, 2024

@yuvipanda - minor docs update:

the CONTRIBUTING.md was missing the steps to npm install and npm run webpack:watch to actually get the form to display on the page.

cc @oliverroick

@yuvipanda
Copy link
Member

@batpad the pip install should have automatically done that (see https://github.com/yuvipanda/jupyterhub-fancy-profiles/blob/5874628f4169afbf66eaa40fbdd96f166e23fda0/setup.py#L31). Did it not?

@batpad
Copy link
Collaborator Author

batpad commented Mar 28, 2024

@yuvipanda that makes sense!

@batpad batpad closed this Mar 28, 2024
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.

2 participants