-
Notifications
You must be signed in to change notification settings - Fork 33
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
When using Filament in PWA mode, the phone fields don't work #66
Comments
I upgraded to latest version (3.1.5) and gived the same error |
Same as in SPA mode too: after refresh, it works fine:
|
Hello guys! I think the async alpine version has been updated and the method of loading the assets called by the phone input has changed. [PR] Can you extend the phone component view and make the following changes and check? I will update according to the situation.
<div
class="w-full"
- x-ignore
@if (FilamentView::hasSpaMode())
- {{-- format-ignore-start --}}ax-load="visible || event (ax-modal-opened)" {{-- format-ignore-end --}}
+ {{-- format-ignore-start --}}x-load="visible || event (ax-modal-opened)" {{-- format-ignore-end --}}
@else
- ax-load
+ x-load
@endif
- ax-load-src="{{ \Filament\Support\Facades\FilamentAsset::getAlpineComponentSrc('filament-phone-input', package: 'ysfkaya/filament-phone-input') }}"
+ x-load-src="{{ \Filament\Support\Facades\FilamentAsset::getAlpineComponentSrc('filament-phone-input', package: 'ysfkaya/filament-phone-input') }}" |
I can confirm this change fixes the current issue. I submitted PR [#67] for this. |
Yeahh! |
Filament PHP version:
3.2.139
ysfkaya/filament-phone-input version:
2.2.7
When using Flament in PWA mode, the fields dont work, but work perfectly on refreshing the page.
The text was updated successfully, but these errors were encountered: