Skip to content

Commit 0c96edc

Browse files
committed
fix(onesignal): make notification contents optional
- Remove requirement for contents in OneSignalRequestBody - Update OneSignalPushNotificationClient to use payload.message instead of payload.title for contents
1 parent 25466da commit 0c96edc

File tree

3 files changed

+3
-4
lines changed

3 files changed

+3
-4
lines changed

lib/src/models/push_notification/onesignal_request_body.dart

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,8 @@ class OneSignalRequestBody extends Equatable {
2020
required this.appId,
2121
required this.includePlayerIds,
2222
required this.headings,
23-
required this.contents,
2423
required this.data,
24+
this.contents,
2525
this.bigPicture,
2626
});
2727

@@ -35,7 +35,7 @@ class OneSignalRequestBody extends Equatable {
3535
final Map<String, String> headings;
3636

3737
/// The notification's content.
38-
final Map<String, String> contents;
38+
final Map<String, String>? contents;
3939

4040
/// The custom data payload
4141
final PushNotificationPayload data;

lib/src/models/push_notification/onesignal_request_body.g.dart

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

lib/src/services/onesignal_push_notification_client.dart

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -103,7 +103,6 @@ class OneSignalPushNotificationClient implements IPushNotificationClient {
103103
appId: appId,
104104
includePlayerIds: deviceTokens,
105105
headings: {'en': payload.title},
106-
contents: {'en': payload.title},
107106
bigPicture: payload.imageUrl,
108107
data: payload,
109108
);

0 commit comments

Comments
 (0)