-
Notifications
You must be signed in to change notification settings - Fork 133
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
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
сейчас 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
тут очень много изменений. Скорее всего из-за отступов в 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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
скорее всего тут нужно указывать не путь к билдам, а путь к ресурсам
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