recipes-contrib
recipes-contrib copied to clipboard
Add V2 of redjan/fcm-bundle
trafficstars
| Q | A |
|---|---|
| License | MIT |
| Packagist | https://packagist.org/packages/redjanym/fcm-bundle |
Thanks for the PR 😍
How to test these changes in your application
-
Define the
SYMFONY_ENDPOINTenvironment variable:# On Unix-like (BSD, Linux and macOS) export SYMFONY_ENDPOINT=https://raw.githubusercontent.com/symfony/recipes-contrib/flex/pull-1747/index.json # On Windows SET SYMFONY_ENDPOINT=https://raw.githubusercontent.com/symfony/recipes-contrib/flex/pull-1747/index.json -
Install the package(s) related to this recipe:
composer req symfony/flex composer req 'redjanym/fcm-bundle:^2.0' -
Don't forget to unset the
SYMFONY_ENDPOINTenvironment variable when done:# On Unix-like (BSD, Linux and macOS) unset SYMFONY_ENDPOINT # On Windows SET SYMFONY_ENDPOINT=
Diff between recipe versions
In order to help with the review stage, I'm in charge of computing the diff between the various versions of patched recipes. I'm going keep this comment up to date with any updates of the attached patch.
redjanym/fcm-bundle
1.1 vs 2.0
diff --git a/redjanym/fcm-bundle/1.1/config/packages/redjan_ym_fcm.yaml b/redjanym/fcm-bundle/2.0/config/packages/redjan_ym_fcm.yaml
index 1d6e668a..ab12591a 100644
--- a/redjanym/fcm-bundle/1.1/config/packages/redjan_ym_fcm.yaml
+++ b/redjanym/fcm-bundle/2.0/config/packages/redjan_ym_fcm.yaml
@@ -1,2 +1,2 @@
redjan_ym_fcm:
- firebase_api_key: '%env(resolve:FCM_SERVER_KEY)%'
+ service_account_file: '%env(resolve:FCM_DEFAULT_SERVICE_ACCOUNT_FILE_PATH)%'
diff --git a/redjanym/fcm-bundle/1.1/manifest.json b/redjanym/fcm-bundle/2.0/manifest.json
index e712b9af..15564cff 100644
--- a/redjanym/fcm-bundle/1.1/manifest.json
+++ b/redjanym/fcm-bundle/2.0/manifest.json
@@ -6,6 +6,6 @@
"config/": "%CONFIG_DIR%/"
},
"env": {
- "FCM_SERVER_KEY": "YOUR_FIREBASE_CLOUD_MESSAGING_SERVER_KEY"
+ "FCM_DEFAULT_SERVICE_ACCOUNT_FILE_PATH": "YOUR_FCM_SERVICE_ACCOUNT_FILE_ABSOLUTE_PATH"
}
}
Hi, @redjanym , are you interested in applying suggested changed? Or we can merge it as is.
Closing as there is no more activities and the current code cannot be merged as is. Feel free to reopen.