search

MihaelIsaev / FCM

⚡️ PushNotifications through FireBase for Vapor 3 and 4.
MIT License
120 stars 33 forks source link

All FCMAndroidConfig fields optional #34

Closed sroebert closed 2 years ago

sroebert commented 2 years ago

All fields in FCMAndroidConfig are suppose to be optional. Since you currently cannot create the FCMAndroidConfig for only setting one particular field, this PR makes all fields optional, allowing to only set certain parameters.

sroebert commented 2 years ago

It works fine yes.

sroebert commented 2 years ago

I reverted back the default value for the priority, the other values are all optional, so these should not change anything for existing library users.

MihaelIsaev commented 2 years ago

@sroebert thank you very much for the contribution! These changes are now available since 2.11.0 tag.