-
Notifications
You must be signed in to change notification settings - Fork 2.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bad confirmation error when saving ad in 30-day ads history #20523
Comments
This is because the server does not recognize |
We have bookmark, isn't that same as saved? |
Changing to |
This can ride the trains in my opinion as this was a bug and never submitted before. @amirsaber @btlechowski your thoughts? |
Verification passed on
Verified test plan from the description. Saved the ad in 30-day history Verified confirmation of type
|
Follow up to #16005
Steps to Reproduce
30-day ads history
frombrave://rewards
note: also reproducible on production
Actual result:
Bad confirmation error in logs
Expected result:
No error
Reproduces how often:
Easily reproduced
Brave version (brave://version info)
cc @brave/legacy_qa @tmancey @jsecretan @rebron @amirsaber
The text was updated successfully, but these errors were encountered: