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

fix: fixes livewire permissions binding #33

Closed
wants to merge 1 commit into from
Closed

fix: fixes livewire permissions binding #33

wants to merge 1 commit into from

Conversation

bellalMohamed
Copy link

Fixes: #32

I did check the documentation of Livewire and actually didn't find anything mentioning wire:model.defer removing .defer or adding both .lazy work fine

@taylorotwell
Copy link
Member

Removing defer is likely not the answer. I will ask Caleb.

@calebporzio
Copy link
Contributor

Ok, this was a bug in Livewire V2.

Adding wire:model.defer didn't work with checkboxes, because Livewire would always use the value of the latest checkbox checked.

Checkboxes are unique to other inputs in that they have multiple elements that combine to form a unified value. Livewire now accounts for this in the following commit: livewire/livewire@68fbb55

If you update Livewire in your application using composer update livewire/livewire, you should be all set 👌

@taylorotwell
Copy link
Member

Thanks!

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.

Bug: API Token permissions aren't bound properly to livewire
3 participants