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

fix assets pipeline in production #36

Closed
wants to merge 1 commit into from
Closed

fix assets pipeline in production #36

wants to merge 1 commit into from

Conversation

point212
Copy link
Contributor

Я так и не смог понять почему же у меня локально всё работает правильно, а на продакшене нужные файлы отсутствуют.

Зато понял что учитывая что у нас Rails 6 и мы используем webpacker для управления ассетами, можно поручить ему компилить scss.

Внес соответствующие изменения.
Попутно еще в конфиге вебпакера разрешил в dev-окружении ему быть более болтливым, чтобы легче было видеть ошибки в asset pipeline

Возможно я сделал полную фигню, и надо было не так. Буду рад любому фидбэку

@point212 point212 changed the title move application.scss under webpacker control fix assets pipeline in production Sep 10, 2019
@mokevnin
Copy link
Member

Саш я не думаю что так стоит делать. Тут ведь автогенерация от шестых рельс, значит подразумевается использования спрокетс все еще. В целом я был бы не против, если бы это была рекомендованная практика для этих рельс.

@point212
Copy link
Contributor Author

Ну окей. Тогда отменяем. Буду дальше копаться и погружаться в дивный мир фронтенда.

А что ты имел в виду под автогенерацией 6-х рельс? В смысле, что если мы сгенерим ещё один контроллер или добавим гем, который влияет на пайплайн - изменения отразятся только в /app/assets?

@mokevnin
Copy link
Member

А что ты имел в виду под автогенерацией 6-х рельс?

То что они по дефолту генерят спрокест файл для css. А ты фактически все это добро удалил.

@mokevnin mokevnin closed this Sep 11, 2019
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.

None yet

2 participants