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.

Sign up for GitHub

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

Jump to bottom

Deprecation of FCM legacy API #28

Closed
eightyfive opened this issue Mar 19, 2024 · 2 comments · Fixed by #31
Closed

Deprecation of FCM legacy API #28

eightyfive opened this issue Mar 19, 2024 · 2 comments · Fixed by #31

Comments

@eightyfive
Copy link

Context:
https://expo.dev/blog/expo-adds-support-for-fcm-http-v1-api

Is there a plan to support the ?useFcmV1=true URL param?
@Brune04
Copy link

Brune04 commented Mar 20, 2024

I've just manually adjusted vendor/yieldstudio/laravel-expo-notifier/src/ExpoNotificationsService.php to add ?useFcmV1=true to the SEND_NOTIFICATION_ENDPOINT variable to cover this right now.

After doing this, it appears that everything works properly. The only issue is when you use shouldBatch your notification icon does not show up on Android. It just shows a blank circle

@JamesHemery
Copy link
Member

@joemugen @dtangdev

joemugen added a commit that referenced this issue Mar 26, 2024
@joemugen
feat: adds support for legacy FPM API use
1d35e28 
Fix #28
@joemugen joemugen mentioned this issue Mar 26, 2024
Merged
JamesHemery pushed a commit that referenced this issue Mar 28, 2024
@joemugen
feat: adds support for legacy FPM API use (#31)
1d93ba4 
Fix #28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: adds support for legacy FPM API use YieldStudio/laravel-expo-notifier
3 participants
@eightyfive @Brune04 @JamesHemery