-
Notifications
You must be signed in to change notification settings - Fork 974
Promotions server should now be polled from payments screen without having payments enabled. #14732
Promotions server should now be polled from payments screen without having payments enabled. #14732
Conversation
app/browser/api/ledger.js
Outdated
appActions.onPromotionGet() | ||
clearTimeout(togglePromotionTimeoutId) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit
Let's wrap this in an if
statement (if (togglePromotionTimeoutId)
)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh! Not a nit for me. Thanks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, test plan works. One comment left
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
++
Enable wallet notification is remaining and stacking with promo notification. Reverting to wip. |
…aving payments enabled. Added tests ✅ Updated clearing timeout with null check. Reworked UGP promo server poll delay. Removed appActions 🔥
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not seeing the stacking issue, lgtm
Promotions server should now be polled from payments screen without having payments enabled.
Promotions server should now be polled from payments screen without having payments enabled.
Addon fix for #14616
Fixes notification breaks from #14621
Submitter Checklist:
git rebase -i
to squash commits (if needed).Test Plan:
LEDGER_VERBOSE=true
v1/grants
).Whether payments is enabled or not, UGP polling should still be present in console when navigating to about:preferences#payments and LEDGER_VERBOSE=true
Reviewer Checklist:
Tests