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

Indefinitely spinning loading graphic due to uncaught null feed parameter #20272

Closed
stephendonner opened this issue Dec 31, 2021 · 2 comments
Closed
Assignees
Labels
closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated.

Comments

@stephendonner
Copy link

Description

Indefinitely spinning loading graphic due to uncaught null feed parameter

Steps to Reproduce

  1. new profile
  2. launch Brave
  3. open a new-tab page
  4. click on Customize
  5. disable all sources via the Brave News card/link
  6. open Developer Tools, Console tab
  7. look at the exception thrown

Actual result:

brave_new_tab.bundle.js:1956 Uncaught Error: Received BraveNewsController_GetFeed_ResponseParams with invalid null field "feed"
    at r.internal.Decoder.decodeStructInline (brave_new_tab.bundle.js:1956:12978)
    at r.internal.interfaceSupport.InterfaceRemoteBase.onMessageReceived (brave_new_tab.bundle.js:1956:36230)
    at r.internal.interfaceSupport.Endpoint.onMessageReceived (brave_new_tab.bundle.js:1956:31278)
    at r.internal.interfaceSupport.Router.onMessageReceived_ (brave_new_tab.bundle.js:1956:28783)
    at r.internal.interfaceSupport.HandleReader.read_ (brave_new_tab.bundle.js:1956:41093)

Screen Shot 2021-12-30 at 4 55 22 PM

Expected result:

No exception; some kind of messaging/UX to remind the user to enable more sources?

Reproduces how often:

100%

Brave version (brave://version info)

Brave 1.36.5 Chromium: 97.0.4692.56 (Official Build) nightly (x86_64)
Revision 04da6c66398ca50e603cc236a07dc7dfd3bbc750-refs/branch-heads/4692@{#990}
OS macOS Version 12.2 (Build 21D5025f)

cc @petemill @mattmcalister @kjozwiak

@mattmcalister mattmcalister added design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. priority/P3 The next thing for us to work on. It'll ride the trains. labels Mar 8, 2022
@jonathansampson
Copy link
Contributor

jonathansampson commented Apr 18, 2022

Note, I am able to reproduce this after enabling only one item.

  1. Disable all sources
  2. Enable Brave Blog alone

Issue reproduces.

The issue is not limited to Brave Blog; other feeds (below is a sample) will also cause this behavior.

✔ 99 BitCoins		https://99bitcoins.com/feed/
✔ 9to5 Mac		https://9to5mac.com/feed/
✔ ABC News		https://abcnews.go.com/abcnews/topstories
✔ Adventure		https://adventure.com/feed/
✔ Ain't It Cool		https://www.aintitcool.com/node/feed/
✔ Al Jazeera		https://www.aljazeera.com/xml/rss/all.xml

❌ Big Think		https://feeds.feedburner.com/bigthink/main
❌ ArseBlog		https://feeds.feedburner.com/arseblog
❌ Android Central	https://www.androidcentral.com/feed
❌ Apartment Therapy	https://www.apartmenttherapy.com/main.rss
❌ Afar Magazine	https://feeds.feedburner.com/AfarMagazine
❌ Brave Blog		https://brave.com/blog/index.xml

@mattmcalister
Copy link

Resolved with Settings upgrade. Closing.

@kjozwiak kjozwiak added closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated. and removed bug design/needs-mock-up needs-mockup A feature which needs design mockup to be implemented. priority/P3 The next thing for us to work on. It'll ride the trains. QA/Yes QA/Test-Plan-Specified OS/Desktop feature/brave-news formerly brave-today labels Jun 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed/stale Issue is no longer relevant, perhaps because the feature it refers to has been deprecated.
Projects
None yet
Development

No branches or pull requests

5 participants