-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
Hook Manage subscription
link in Settings
dialog up to the Accounts site
#22694
Comments
Looks like we were going to include this as part of #18467 (comment) but have since punted, leaving |
I think @nullhook has a PR for this somewhere (hooking up the rest of the dialog UI, am I right?), but having explicit issues to verify individual changes is always good. |
This is thrown in the console:
|
Manage subscription
link in Settings
dialog up to the Accounts site
@spylogsster looks like you already fixed this. Was it fixed along with #22874? |
fixed by brave/brave-core#13348 |
Verified
|
Brave | 1.41.55 Chromium: 102.0.5005.115 (Official Build) nightly (x86_64) |
---|---|
Revision | 174dbe6e33bc81994fceb71d751be201d0b4803d-refs/branch-heads/5005_109@{#3} |
OS | macOS Version 12.5 (Build 21G5037d) |
Followed the steps to reproduce from #22694 (comment).
Confirmed clicking on Manage Subscription
on the VPN
dialog loads https://account.brave.software/account/
, when using the development
environment.
brave://flags |
Manage Subscription link |
https://account.brave.software/account/ |
---|---|---|
Verification
Followed the steps to reproduce from #22694 (comment). Confirmed clicking on
|
Description
Hook
Manage subscription
link in Settings dialog up to the accounts siteSteps to Reproduce
1.39.91
https://bravesoftware.slack.com/archives/CC5SA8CCB/p1651597201141019?thread_ts=1651597077.189229&cid=CC5SA8CCB
VPN
buttonSettings
(gears) buttonManage subscription
Actual result:
Nothing happens
Expected result:
IIRC it should go to (at least on
development
)https://account.brave.software/
(account.brave.com
in prod)Reproduces how often:
Brave version (brave://version info)
Version/Channel Information:
no
yes
yes
cc @nullhook @bsclifton @brave/qa-team
The text was updated successfully, but these errors were encountered: