Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Limit 24 hour payment notification to once per reconcile period #5296

Merged
merged 1 commit into from
Oct 31, 2016

Commits on Oct 31, 2016

  1. Limit 24 hour payment notification to once per reconcile period

    Fix #4481
    
    Auditors: @bsclifton
    
    Test Plan:
    1. Trigger the "Payment in 24 hours, please review" notification.
      - Update reconcileStamp to <24 hours from now
      - Have sufficient funds; OR disable the sufficient funds conditional by adding false to ledger.js L1485 (showEnabledNotifications())
      - Change startup notification delay in ledger.js L484 from 15m to 5s
    2. Open Brave and observe 24h review notification
    3. Close Brave and reopen. Notification should not reappear.
    (Next 24h notification timestamp is set in Application Support/brave-development/session-store-1)
    ayumi committed Oct 31, 2016
    Configuration menu
    Copy the full SHA
    addfd70 View commit details
    Browse the repository at this point in the history