add migrate Laravel Mix to Vite #1627
Conversation
Makefile
Outdated
| @@ -10,7 +10,7 @@ deploy: | |||
| git push heroku main | |||
|
|
|||
| setup: env-prepare sqlite-prepare install key db-prepare ide-helper | |||
| npm run development | |||
| npm run dev | |||
There was a problem hiding this comment.
сейчас npm run dev запускает сервер статики, но этого не должно быть. make setup это команда сборки проекта.
тут скорее должен быть билд ассетов, Как это и раньше делается. А старт веб-сервера vite отдельно
Сейчас из-за этого сломана сборка https://github.com/Hexlet/hexlet-sicp/actions/runs/9333045976/job/25689616359?pr=1627
| "engines": { | ||
| "node": "20.x" | ||
| } | ||
| "private": true, |
There was a problem hiding this comment.
тут очень много изменений. Скорее всего из-за отступов в 4 символа. Подключи editorconfig, чтобы он автоматом подтянул нужные настройки отступов в файлах
| @@ -6,7 +6,7 @@ | |||
| <code>b</code> | |||
| {{ __('exercises/1_29.description.4') }} | |||
| </p> | |||
| <img class="img-fluid" src="{{ mix('images/exercises/1_29.gif') }}" alt="1.29"> | |||
| <img class="img-fluid" src="{{ asset('build/images/exercises/1_29.gif') }}" alt="1.29"> | |||
There was a problem hiding this comment.
скорее всего тут нужно указывать не путь к билдам, а путь к ресурсам
https://laravel.com/docs/11.x/vite#blade-processing-static-assets
<img src="{{ Vite::asset('resources/images/logo.png') }}">, возможно даже через директиву @vite это делается.
Щас я еще сам чекну.
|
@DmitriySmolin я внес изменения, но не могу запушить тебе в ветку. Можешь взять их из ветки, два коммита |
|
@fey Все теперь проект собрался! Спасибо за подсказу по интеграции картинок! |
|
@DmitriySmolin не все изменения взял. Там было два коммита, в одном из них я пофиксил сетап для работы с докер композом. |
|
@fey Понял, сегодня разверную проект в Docker и постараюсь исправить развертывание в силу своих возможностей! |
|
@fey Обновил конфигурацию docker и vite. Теперь можно получить ассеты и стили! |
Pull request details
Issues fixed
Link to demo
https://hexlet-sicp-wums.onrender.com/ru