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

Migration laravel 9 above version #5801

Open
wants to merge 2 commits into
base: 1.6
Choose a base branch
from

Conversation

bhargavraviya
Copy link

  • change migration class for future laravel version 9 to 11
  • change to class
  • change some facades in migration files

@tnylea and @emptynick

@jonathanribas
Copy link

jonathanribas commented Aug 20, 2024

Hi,
Laravel 10 bug fixes are over since 7th August 2024, PHP 8.2 Security fixes until February 2025.
Any chance to have this one moving forward?
Thanks in advance!

@bhargavraviya
Copy link
Author

yes

@jonathanribas

Please check this line of code in your codebase

https://github.com/thedevdojo/voyager/pull/5801/files

in this line
resources/views/bread/browse.blade.php

@soinalastudio
Copy link

@bhargavraviya Better way is to fork and start over there as I think there will be any update for voyager from devdojo. They left voyager and in the new version of their product Wave, they will use Filament. You can see in the branch v.3 of wave, Voyager is not used anymore. We will follow you for the fork to have laravel 11 support.

@soinalastudio
Copy link

@bhargavraviya Can we have your fork working for Laravel11?

@bhargavraviya
Copy link
Author

@bhargavraviya Can we have your fork working for Laravel11?

yes it working on laravel 11
please check this PR #5878

@soinalastudio
Copy link

soinalastudio commented Sep 15, 2024 via email

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.

3 participants