🐛 Bug Report: FCM data-type notification expect a message property in the payload

[cherfia]

📜 Description

Currently, when sending a data-type notification, it lacks the push message content that was initially set during the push provider configuration step in the workflow. After inspecting the code below

novu/providers/fcm/src/lib/fcm.provider.ts

Line 69 in bfc3862

data: { ...payload, title: options.title, body: options.content },

It turned out that it uses the body property instead of message.

According to Expo documentation, as stated in Send notifications with FCM & APNs, the correct property to use should be message rather than body. I further confirmed this by introducing a custom field labeled message to the payload given that the data property essentially combines the different payload and overrides properties. It worked, however, the drawback is that the message property originated from the backend, which implies that we would be unable to configure it within the workflow, templates, and internationalization (i18n).

👟 Reproduction steps

curl --location --request POST 'https://eu.api.novu.co/v1/events/trigger' \
--header 'Authorization: ApiKey <REPLACE_WITH_API_KEY>' \
--header 'Content-Type: application/json' \
--data-raw '{
         "name": "space-membership-request",
         "to": {
           "subscriberId": "4e4bcfe9-1b9e-4383-9784-7d36572adfd3"
         },
         "payload": {
           "data": {
             "user": {
               "id": "345a91b3-941b-4274-8e33-49485c4d7cda",
               "full_name": "Luigi",
             },
             "space": {
               "id": "c1c1989b-1b51-489b-9219-57274a8d65cb",
               "name": "super-mario-bros-wonder",
               "title": "Super Mario Bros. Wonder",
             },
             "request_message": "Mama mia!",
             "request_datetime": "2023-10-23T14:55:58.976406"
           },
           "notification_type": "SPACE_MEMBERSHIP_REQUEST",
           "notification_version": "1.0.0"
         },
         "overrides": {
          "type": "data",
           "fcm": {
            "type": "data"
           }
         },
         "tenant": null
}'

👍 Expected behavior

👎 Actual Behavior with Screenshots

Novu version

Novu SaaS

npm version

No response

node version

No response

📃 Provide any additional context for the Bug.

No response

👀 Have you spent some time to check if this bug has been raised before?

• I checked and didn't find a similar issue

🏢 Have you read the Contributing Guidelines?

• I have read the Contributing Guidelines

Are you willing to submit PR?

Yes I am willing to submit a PR!

[cherfia]

Resolved via #4652.