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

brave://memories link in brave://aboutdoesn't work #15582

Closed
stephendonner opened this issue Apr 30, 2021 · 4 comments · Fixed by brave/brave-core#9835
Closed

brave://memories link in brave://aboutdoesn't work #15582

stephendonner opened this issue Apr 30, 2021 · 4 comments · Fixed by brave/brave-core#9835
Assignees
Labels
bug OS/Desktop priority/P5 Not scheduled. Don't anticipate work on this any time soon. QA Pass-macOS QA/Yes release-notes/exclude

Comments

@stephendonner
Copy link

stephendonner commented Apr 30, 2021

Description

brave://memories link in brave://aboutdoesn't work

Steps to Reproduce

  1. load brave://about/
  2. enable the feature flag via brave://flags
  3. click on chrome://memories, which redirects internally to brave://memories/

Actual result:

Broken internal link

example example example
Screen Shot 2021-04-30 at 8 19 06 AM Screen Shot 2021-04-30 at 11 13 32 AM Screen Shot 2021-04-30 at 8 18 38 AM

Expected result:

Working link, or for it to be removed

Reproduces how often:

100%

Brave version (brave://version info)

Brave 1.24.81 Chromium: 90.0.4430.93 (Official Build) (x86_64)
Revision 4df112c29cfe9a2c69b14195c0275faed4e997a7-refs/branch-heads/4430@{#1348}
OS macOS Version 11.3 (Build 20E232)
@rebron rebron self-assigned this May 3, 2021
@rebron rebron added the needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. label May 3, 2021
@bsclifton
Copy link
Member

@stephendonner can you try enabling brave://flags/#memories and then seeing if it works?

More info here:
https://www.ghacks.net/2021/04/24/google-is-working-on-a-new-browsing-history-interface-called-memories-in-chrome/

@stephendonner
Copy link
Author

@bsclifton shortly after @rebron told me it requires a flag, in Slack, I amended the steps and screenshots here to reflect that I re-tested with that feature flag enabled. Confirming that it still isn't displaying anything for me with

Brave 1.26.6 Chromium: 90.0.4430.93 (Official Build) nightly (x86_64)
Revision 4df112c29cfe9a2c69b14195c0275faed4e997a7-refs/branch-heads/4430@{#1348}
OS macOS Version 11.3.1 (Build 20E241)

@rebron rebron added the priority/P5 Not scheduled. Don't anticipate work on this any time soon. label May 7, 2021
@bsclifton bsclifton assigned bsclifton and unassigned rebron Aug 24, 2021
@bsclifton bsclifton assigned goodov and unassigned bsclifton Aug 25, 2021
@bsclifton bsclifton removed the needs-discussion Although the issue is clear, we haven't yet reached a decision about the right solution. label Aug 25, 2021
@goodov goodov added this to the 1.30.x - Nightly milestone Aug 25, 2021
@stephendonner
Copy link
Author

Verified PASSED using

Brave 1.30.56 Chromium: 93.0.4577.58 (Official Build) beta (x86_64)
Revision c4410ece044414ea42fa4ba328d08195e818a99c-refs/branch-heads/4577@{#1076}
OS macOS Version 11.5.2 (Build 20G95)

Steps:

  1. new profile
  2. launched Brave
  3. opened brave://about
  4. looked for brave://memories

Confirmed that brave://memories is now removed from brave://about; however it's still present in brave://flags

Screen Shot 2021-08-30 at 2 33 29 PM

@goodov should we also remove the two memories items in brave://flags, in a separate issue?

example example example
Screen Shot 2021-08-30 at 2 34 06 PM Screen Shot 2021-08-30 at 2 37 24 PM Screen Shot 2021-08-30 at 2 37 32 PM

@goodov
Copy link
Member

goodov commented Aug 31, 2021

@goodov should we also remove the two memories items in brave://flags, in a separate issue?

why not, let's clean up this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug OS/Desktop priority/P5 Not scheduled. Don't anticipate work on this any time soon. QA Pass-macOS QA/Yes release-notes/exclude
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants