Skip to content
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

add funds link on tip banner yields no action - follow up to 2628 #3227

Closed
LaurenWags opened this issue Feb 5, 2019 · 5 comments · Fixed by brave/brave-core#1580
Closed

Comments

@LaurenWags
Copy link
Member

Description

Follow up to #2628

Clicking on add funds link in site tipping banner doesn't do anything. It should link to brave://rewards/#add-funds as the panel does.

Steps to Reproduce

  • Clean install
  • enable rewards
  • Visit site and open rewards panel
  • Click Send a tip...
  • Choose tip amount
  • Click add funds link

Actual result:

Nothing happens
screen shot 2019-02-05 at 1 25 30 pm

Expected result:

Should link to brave://rewards/#add-funds
screen shot 2019-02-05 at 1 27 16 pm

Reproduces how often:

easy

Brave version (brave://version info)

Brave 0.60.19 Chromium: 72.0.3626.81 (Official Build) beta(64-bit)
Revision ac8b982e05014492d1bd7d317628a4f22a97ffa0-refs/branch-heads/3626@{#796}
OS Mac OS X

Reproducible on current release:

  • Does it reproduce on brave-browser dev/beta builds? yes

Website problems only:

  • Does the issue resolve itself when disabling Brave Shields? n/a
  • Is the issue reproducible on the latest version of Chrome? n/a

Additional Information

@NejcZdovc
Copy link
Contributor

link is there, but get this error in the console

image

If I have it like this:
image
then we get an error. But if I change it to
image
then it works.

cc @bbondy @bridiver did we change some permissions where we only allow chrome links?

@NejcZdovc
Copy link
Contributor

@LaurenWags so link is there and all code is correct. At some point we apparently changed permissions and brave links are blocked in webui

@NejcZdovc
Copy link
Contributor

this was changed with brave/brave-core#1196, so will create PR to change from brave to chrome

@GeetaSarvadnya
Copy link

Reproducible on windows 10 x64 - 0.60.19 -Getting console error, when click on add funds link:

image

@LaurenWags
Copy link
Member Author

LaurenWags commented Feb 8, 2019

Verified passed with

Brave 0.60.23 Chromium: 72.0.3626.96 (Official Build) beta(64-bit)
Revision 84098ee7ef8622a9defc2ef043cd8930b617b10e-refs/branch-heads/3626@{#836}
OS Mac OS X

Verification passed on

Brave 0.60.23 Chromium: 72.0.3626.96 (Official Build) beta (64-bit)
Revision 84098ee7ef8622a9defc2ef043cd8930b617b10e-refs/branch-heads/3626@{#836}
OS Windows 10

Verification passed on

Brave 0.60.26 Chromium: 72.0.3626.96 (Official Build) beta(64-bit)
Revision 84098ee7ef8622a9defc2ef043cd8930b617b10e-refs/branch-heads/3626@{#836}
OS Linux

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment