This repository has been archived by the owner on Dec 11, 2019. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 974
Ledger notifications for adding funds should only be shown if brave payments is switched on #5441
Comments
See also: |
I'm not able to reproduce. Steps I tried:
Can you provide repro steps? |
@bbondy - i can't reproduce this either. i found one small bit of code that was logically correct -- but redundant -- which i simplified. can you reproduce it on the current master? thanks! |
I don't know how to reproduce, my wife just got the notification on her computer, and I checked preferences and the Payments master switch was turned off. There have been fixes since then though, not sure if it would be fixed now or not. |
moved to backlog until someone has steps. Feel free to close if you think it's fixed by something else. |
This was referenced Dec 12, 2016
This was referenced Dec 20, 2016
Test plan: #5721 |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
I noticed it said it was time to add funds but I did not have Brave Payments enabled.
The text was updated successfully, but these errors were encountered: