Skip to content

Commit

Permalink
fix(subscriptions): disable notification limit
Browse files Browse the repository at this point in the history
  • Loading branch information
lirantal committed Dec 7, 2019
1 parent c9503f1 commit 22b30a4
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions functions/src/Subscriptions.service.js
Original file line number Diff line number Diff line change
Expand Up @@ -161,13 +161,13 @@ class Subscriptions {

// @FIXME also consider if we actually want to limit it? or just
// to have a flag whether it was ever notified or not
if (
subscriptionItem &&
subscriptionItem.notified &&
subscriptionItem.notified === true
) {
throw new Error("Subscription token already notified");
}
// if (
// subscriptionItem &&
// subscriptionItem.notified &&
// subscriptionItem.notified === true
// ) {
// throw new Error("Subscription token already notified");
// }

if (!subscriptionItem.subscription.endpoint) {
throw new Error("Malformed subscription object");
Expand Down

0 comments on commit 22b30a4

Please sign in to comment.