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

Advanced setting button should be hidden when payments is disabled #4790

Closed
Sh1d0w opened this issue Oct 14, 2016 · 6 comments · Fixed by #4801
Closed

Advanced setting button should be hidden when payments is disabled #4790

Sh1d0w opened this issue Oct 14, 2016 · 6 comments · Fixed by #4801

Comments

@Sh1d0w
Copy link

Sh1d0w commented Oct 14, 2016

Did you search for similar issues before submitting this one?

No.

Describe the issue you encountered:
Clicking on the "Advanced settings" button while payments are turned off produces no action. If you turn payments on then a pop up appears.

Expected behavior:

When payments are off, the button should be hidden, if it will not trigger any action.

  • Platform (Win7, 8, 10? macOS? Linux distro?): All
  • Brave Version: 0.12.5-1
  • Steps to reproduce:
    1. Go to payments
    2. Turn them off
    3. Click advanced settings
  • Screenshot if needed:
  • Any related issues:
@Sh1d0w Sh1d0w added this to the 0.12.5dev milestone Oct 14, 2016
@bsclifton bsclifton changed the title Advansed setting button in payments page Advanced setting button should be hidden when payments is disabled Oct 14, 2016
@luixxiul
Copy link
Contributor

I cannot confirm this was fixed on 0.12.5 RC2, on Windows 10. Regression or something?

@luixxiul luixxiul reopened this Oct 16, 2016
@luixxiul luixxiul added the needs-investigation A bug not 100% confirmed/fixed that needs QA to better audit. label Oct 16, 2016
@bbondy bbondy modified the milestones: 0.12.6dev, 0.12.5dev Oct 16, 2016
@bbondy
Copy link
Member

bbondy commented Oct 17, 2016

This is not in 0.12.5 but it should be fixed on master 0.12.6.
By the way you can see the tags it is on by clicking on the commit, the one we do builds from is dev-channel.

@bbondy bbondy closed this as completed Oct 17, 2016
@luixxiul
Copy link
Contributor

I thought this is it, which has been tagged 0.12.5 RC builds: 7666907

@luixxiul luixxiul removed the needs-investigation A bug not 100% confirmed/fixed that needs QA to better audit. label Oct 17, 2016
@bsclifton
Copy link
Member

ah- definitely my bad assuming it was pulled into 0.12.5

@luixxiul
Copy link
Contributor

luixxiul commented Nov 1, 2016

This is happening again on the latest master.

@bsclifton
Copy link
Member

@luixxiul can you open a new issue w/ steps to reproduce? (and link back here) Thanks 😄

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants