Nits, added early return in sendUnique if attribution type is Not Supp

src/OneSignal.ts

@@ -839,6 +839,12 @@ export default class OneSignal {
       return;
     }
     const outcomeAttribution = await outcomesHelper.getAttribution();
+
+    if (outcomeAttribution.type === OutcomeAttributionType.NotSupported) {
+      Log.warn("You are on a free plan. Please upgrade to use this functionality.");
+      return;
+    }
+
     // all notifs in attribution window
     const { notificationIds } = outcomeAttribution;
     // only new notifs that ought to be attributed

src/helpers/shared/OutcomesHelper.ts

@@ -97,8 +97,7 @@ export default class OutcomesHelper {
 
   shouldSendUnique(outcomeAttribution: OutcomeAttribution, notifArr: string[]) {
     // we should only send if type is unattributed OR there are notifs to attribute
-    if (outcomeAttribution.type === OutcomeAttributionType.Unattributed ||
-        outcomeAttribution.type === OutcomeAttributionType.NotSupported) {
+    if (outcomeAttribution.type === OutcomeAttributionType.Unattributed) {
       return true;
     }
     return notifArr.length > 0;

src/models/OutcomeProps.ts

@@ -4,4 +4,4 @@ export default interface OutcomeProps {
   type: OutcomeAttributionType;
   notificationIds: string[];
   weight?: number|undefined;
-}
+}

test/unit/public-sdk-apis/sendUniqueOutcome.ts

@@ -238,4 +238,4 @@ test("attribution of an outcome with multiple notifications happens correctly",
 
   const sentOutcome = await Database.get<SentUniqueOutcome>("SentUniqueOutcome", OUTCOME_NAME);
   t.true(_.isEqual(sentOutcome.notificationIds, notificationArr));
-});
+});