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

Rename nav.js to Nav.js #3

Merged
merged 1 commit into from
Mar 14, 2016
Merged

Rename nav.js to Nav.js #3

merged 1 commit into from
Mar 14, 2016

Conversation

LucasFrecia
Copy link
Contributor

Renaming file to avoid following error when running webpack:

ERROR in ./src/js/pages/Layout.js
Module not found: Error: Cannot resolve 'file' or 'directory' ../components/layout/Nav in /var/www/html/react-js-tutorials/2-react-router/src/js/pages
@ ./src/js/pages/Layout.js 20:11-46

Renaming file to avoid following error when running webpack:

ERROR in ./src/js/pages/Layout.js
Module not found: Error: Cannot resolve 'file' or 'directory' ../components/layout/Nav in /var/www/html/react-js-tutorials/2-react-router/src/js/pages
 @ ./src/js/pages/Layout.js 20:11-46
@moki
Copy link

moki commented Mar 14, 2016

22 days ago...
no idea why its not merged yet.

@Sudakatux
Copy link

Seems to solve a simple mistake. Please merge

@willrstern
Copy link
Collaborator

Ah! Fell into the email black hole! Merging.

willrstern added a commit that referenced this pull request Mar 14, 2016
@willrstern willrstern merged commit 9c05714 into learncodeacademy:master Mar 14, 2016
@moki
Copy link

moki commented Mar 14, 2016

@willrstern thanks :)

helmutgranda added a commit to helmutgranda/react-js-tutorials that referenced this pull request Oct 1, 2016
brennan-karrer pushed a commit to brennan-karrer/react-js-tutorials that referenced this pull request Feb 24, 2017
NimmiW pushed a commit to NimmiW/react-js-tutorials that referenced this pull request Mar 13, 2017
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.

4 participants