Duplicated top site entries after update
See original GitHub issueDescription
After updating to the last nightly and opening Google Translate from top sites bar, this bug occurred. Instead of my sites, there were only pinned ones. In addition, the GitHub icon has disappeared (perhaps the disappearance of the GitHub icon is due to its updating?).
Steps to Reproduce
- Update to last nightly
- Open something from top sites bar
Actual result:
Expected result:
List of my top sites and correct displaying of GitHub icon.
Reproduces how often:
N/A
Brave version (brave://version info)
Brave: 1.9.1 Chromium: 81.0.4044.83 (Official build) unknown (64-bit)
Version: 5718ca617723806b9aacfb41108a0d911d1bde74-refs/branch-heads/4044@{#853}
OS: Linux
Version/Channel Information:
- Q: Can you reproduce this issue with the nightly channel? A: Yes
Other Additional Information:
- Q: Does the issue resolve itself when disabling Brave Shields? A: No
- Q: Does the issue resolve itself when disabling Brave Rewards? A: No
- Q: Is the issue reproducible on the latest version of Chrome? A: Yes
Issue Analytics
- State:
- Created 3 years ago
- Comments:5 (4 by maintainers)
Top Results From Across the Web
Duplicate Content: Why does it happen and how to fix issues
When duplicate content is present, site owners can suffer rankings and traffic losses. These losses often stem from two main problems: To provide...
Read more >Duplicate content: Causes and solutions - Yoast
Duplicate content is content which is available on multiple URLs on the web. Because more than one URL shows the same content, search...
Read more >Troubleshoot: Duplicate articles or landing pages appearing ...
If you are experiencing duplicate article or landing page entries in your Chartbeat data, either in your Real-Time or Historical dashboard ...
Read more >Deftly dealing with duplicate content - Google Developers
How can Webmasters proactively address duplicate content issues? · Block appropriately: · Use 301 redirects: · Be consistent: · Use TLDs: · Syndicate...
Read more >Update one of 2 duplicates in an sql server database table
I think you can use TOP() operator instead of row_number() method it will help ... with RRN function you can update the last...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
Thanks @congard, with the data you shared I was able to reproduce and just opened a pull-request.
Verification PASSED on
macOS 10.15.4 x64 Catalina
using the following build:Went through the cases outlined under https://github.com/brave/brave-core/pull/5173#issue-400312677 and https://github.com/brave/brave-core/pull/5173#issuecomment-617724800 x2 times to ensure everything was working as expected.
Populated several tiles on the NTP using
1.7.98 CR: 81.0.4044.113
as per the following:Updated to
1.8.86 CR: 81.0.4044.129
and ensured that I didn’t lose any tiles on the NTP:Used the script from https://github.com/brave/brave-core/pull/5173#issue-400312677 and ensured that both
https://translate.google.com
andhttps://github.com/congard
where added in the correct positions:Verification passed on
Went through the cases outlined under https://github.com/brave/brave-core/pull/5173#issue-400312677 and https://github.com/brave/brave-core/pull/5173#issuecomment-617724800 and ensure everything was working as expected.
Populated several tiles on the NTP using
1.7.98 CR: 81.0.4044.113
as per the following:Updated to
1.8.86 CR: 81.0.4044.129
and ensured that I didn’t lose any tiles on the NTP:Used the script from https://github.com/brave/brave-core/pull/5173#issue-400312677 and ensured that both
https://translate.google.com
andhttps://github.com/congard
where added in the correct positions:Verification passed on
Upgraded from 1.7.98. Verified no duplicate top sites. Before upgrade: After upgrade
Verified test plan from https://github.com/brave/brave-core/pull/5173 Used the script to generate duplicate top sites.