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

Make Gretel embrace Inertia v2 #20

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

onlime
Copy link

@onlime onlime commented Nov 23, 2024

Current Gretel at school age of v1.8.0 does not like to play with Inertia v2. Let's tell her a fairy tale, where Inertia v2 plays Hänsel and they push the witch into the oven, so they can find their safe way back home following the bread crumbs.

... as it does not sound like inertiajs/inertia-laravel#678 is going to get merged.

@onlime
Copy link
Author

onlime commented Dec 8, 2024

Hi @inxilpro can you please look into this / merge it, as it now also includes a PHP 8.4 compatibility fix in 524b365, bumping laravel/serializable-closure to 2.x 🙏

@onlime
Copy link
Author

onlime commented Dec 13, 2024

I have reverted my initial commit in 107993a, as this issue was fixed in Inertia v2.0.0.

But we would still need to bump laravel/serializable-closure to v2.0

@onlime
Copy link
Author

onlime commented Dec 18, 2024

Hi @inxilpro is this package still maintained? do you want me to help out?

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.

1 participant