Skip to content

fix: дать возможность публиковать миграции#5

Open
SomeRandomGithubUserXd wants to merge 3 commits intomoonshine-software:2.xfrom
SomeRandomGithubUserXd:two-factor-migrtaion-fix
Open

fix: дать возможность публиковать миграции#5
SomeRandomGithubUserXd wants to merge 3 commits intomoonshine-software:2.xfrom
SomeRandomGithubUserXd:two-factor-migrtaion-fix

Conversation

@SomeRandomGithubUserXd
Copy link

В нашем проекте нет таблицы moonshine_users, поэтому в момент, когда ларавель ищет сервис провайдеры в пакетах и подгружает из них миграции - вылезает неустранимая ошибка.
Надо дать возможность опубликовать миграцию и сменить название таблицы на кастомное

@lee-to
Copy link
Member

lee-to commented Oct 8, 2025

в 11 или 12 версии появился метод для публикации - https://laravel.com/docs/12.x/packages#migrations, может прочекать если он есть то юзать его а если нет то по старинке

@SomeRandomGithubUserXd
Copy link
Author

Все равно из провайдеры вызывается метод, подгружающий пакетную миграцию - его то точно надо убрать

@SomeRandomGithubUserXd
Copy link
Author

SomeRandomGithubUserXd commented Oct 8, 2025

А так да, ты прав - поменял на publishesMigrations, они подстроятся в таком случае под дату паблиша - есть смысл. Есть для 11 и 12 версии

@lee-to
Copy link
Member

lee-to commented Oct 12, 2025

Совместимость с 10 нарушится, надо проверять на method_exists или версию

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