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

added missing dep in bower and added config.js in brunch #4

Open
wants to merge 2 commits into
base: feature/newleafng
Choose a base branch
from

Conversation

freddylimpens
Copy link
Contributor

@glibersat j'ai eu besoin de faire ces modifs pour pouvoir lancer l'appli.

@glibersat
Copy link
Member

config.js ne devrait pas être dans brunch, sinon ça écrase ton fichier à chaque fois ! Il faut mettre un template de config, mais pas le fichier.

@freddylimpens
Copy link
Contributor Author

Hum, mais comment brunch va chercher le fichier js/config.js qui est appelé dans le index.html (l.85):

<!-- app -->
    <script src="js/config.js"></script>
    <script src="js/app.js"></script> 

Comment fais-tu de ton côté ?

@glibersat
Copy link
Member

Comme son nom l'indique, c'est un fichier de config, donc c'est au user
de le poser dans _public/js/config.js

Le 18/05/2015 11:16, freddylimpens a écrit :

Hum, mais comment brunch va chercher le fichier |js/config.js| qui est
appelé dans le index.html (l.85):

|
<script src="js/config.js"></script>
<script src="js/app.js"></script>
|

Comment fais-tu de ton côté ?


Reply to this email directly or view it on GitHub
#4 (comment).

http://sigill.org

@freddylimpens
Copy link
Contributor Author

ok, understood :) Je corrige mon PR en fonction. thks

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