YieldStudio / laravel-expo-notifier

Easily manage Expo notifications with Laravel. Support batched notifications.
MIT License
33 stars 11 forks source link

useFcmV1 must be a boolean error #33

Open Brune04 opened 4 months ago

Brune04 commented 4 months ago

After upgrading to 0.0.14 I'm now getting this error:

YieldStudio\LaravelExpoNotifier\Exceptions\ExpoNotificationsException: Expo service error: HTTP 400 response: {"errors":[{"code":"VALIDATION_ERROR","message":"\"useFcmV1\" must be a boolean.","isTransient":false,"requestId":"6471d747-14dc-4caf-a9e1-0388

This is what my expo-notifications.php config looks like: 'use_fcm_legacy_api' => env('EXPO_USE_FCM_LEGACY_API', false),

and this is what I have in my .env EXPO_USE_FCM_LEGACY_API=true

I also tested just putting true and just putting false in the expo-notification.php file. It works fine whenever I put false, but not whenever I put true.

joemugen commented 4 months ago

Thanks for the assignment as, after a quick check, the problem seems more complicated than expected 😓 I'll give a look ASAP.

From what I tested, the useFcmV1 is a query param with the value true or false but as a string (since it's a query param). I don't know why we'd get this validation error then... Hope to have good news soon.