-
Notifications
You must be signed in to change notification settings - Fork 68
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
Not generating custom asset when default_lang
!= first language
#233
Comments
default_lang
!= en-us
default_lang
!= first language
Update, when I change the order of the languages in the
languages: ["pt-br", "en-us"]
default_lang: "pt-br" It works for ls _site/assets/js/
cookies.js levenshtein.js pwa.js scriptsss.js theme.js
ls _site/en-us/assets/js/
scriptsss.js but now, it doesn't work for
languages: ["pt-br", "en-us"]
default_lang: "en-us" ls _site/assets/js/
cookies.js levenshtein.js pwa.js theme.js
ls _site/pt-br/assets/js/
scriptsss.js Meaning, it only correctly generates the file if the |
Since it works when the first language is the
How do you want to proceed with this? |
Great investigation! and excellent clues. defs this is interesting behavior.
The list of all site languages to
this process happens before site.prepare occurs. Although the true order is entirely by documents, if a document doesn't specify a all tests around this have the default_lang as the first language in the site configuration. I'll try and think of more things. I can imagine some tests to write around these that could be easy to verify. |
Should I close this as completed? |
have you tried using the latest changes not yet released (run
|
Here is the situation:
_config.yml
_scripts/scripts.js.liquid
It correctly creates
Everything is fine,
scriptsss.js
was correctly created in the assets dir for every language.Now, if I change only the
default_lang
in_config.yml
topt-br
, this happens:Any idea why it doesn't create the file
_site/assets/js/scriptsss.js
?The text was updated successfully, but these errors were encountered: