Skip to content

Commit

Permalink
[Brave News]: Fix bug where explicitly disabled sources would be migr…
Browse files Browse the repository at this point in the history
…ated (#15608)
  • Loading branch information
fallaciousreasoning authored Oct 24, 2022
1 parent 7e72d27 commit 1ac5fef
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion components/brave_today/browser/publishers_controller.cc
Original file line number Diff line number Diff line change
Expand Up @@ -174,7 +174,10 @@ void PublishersController::EnsurePublishersIsUpdating() {
<< publisher_id
<< ". This could be because we've removed the publisher. "
"Attempting to migrate to a direct feed.";
missing_publishers_.push_back(publisher_id);
// We only care about missing publishers if the user was subscribed
// to them.
if (is_user_enabled.value_or(false))
missing_publishers_.push_back(publisher_id);
}
}
// Add direct feeds
Expand Down

0 comments on commit 1ac5fef

Please sign in to comment.