recipes-contrib icon indicating copy to clipboard operation
recipes-contrib copied to clipboard

Add V2 of redjan/fcm-bundle

Open redjanym opened this issue 9 months ago • 2 comments
trafficstars

Q A
License MIT
Packagist https://packagist.org/packages/redjanym/fcm-bundle

redjanym avatar Jan 27 '25 23:01 redjanym

Thanks for the PR 😍

How to test these changes in your application

  1. Define the SYMFONY_ENDPOINT environment 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
    
  2. Install the package(s) related to this recipe:

    composer req symfony/flex
    composer req 'redjanym/fcm-bundle:^2.0'
    
  3. Don't forget to unset the SYMFONY_ENDPOINT environment 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"
     }
 }

github-actions[bot] avatar Jan 27 '25 23:01 github-actions[bot]

Hi, @redjanym , are you interested in applying suggested changed? Or we can merge it as is.

diimpp avatar Feb 10 '25 16:02 diimpp

Closing as there is no more activities and the current code cannot be merged as is. Feel free to reopen.

fabpot avatar Aug 21 '25 06:08 fabpot