Issue #11605: Add option to include provided top sites in TopSitesConfig #11608

gabrielluong · 2022-01-25T20:21:14Z

Quality: This PR builds and passes detekt/ktlint checks (A pre-push hook is recommended)
Tests: This PR includes thorough tests or an explanation of why it does not
Changelog: This PR includes a changelog entry or does not need one
Accessibility: The code in this PR follows accessibility best practices or does not include any user facing features

Milestone: Make sure issues closed by this pull request are added to the milestone of the version currently in development.
Breaking Changes: If this is a breaking change, please push a draft PR on Reference Browser to address the breaking issues.

…in TopSitesConfig

gabrielluong · 2022-01-25T20:36:53Z

This is the followup to the breaking change mozilla-mobile/fenix#23409

MozillaNoah

LGTM

gabrielluong added the 🕵️‍♀️ needs review PRs that need to be reviewed label Jan 25, 2022

gabrielluong requested review from Amejia481 and MozillaNoah January 25, 2022 20:21

gabrielluong requested a review from grigoryk as a code owner January 25, 2022 20:21

Issue mozilla-mobile#11605: Add option to include provided top sites …

cd0f1ce

…in TopSitesConfig

gabrielluong force-pushed the 11605 branch from b09803e to cd0f1ce Compare January 25, 2022 20:23

gabrielluong removed the request for review from grigoryk January 25, 2022 20:36

MozillaNoah approved these changes Jan 25, 2022

View reviewed changes

gabrielluong added 🛬 needs landing PRs that are ready to land and removed 🕵️‍♀️ needs review PRs that need to be reviewed labels Jan 25, 2022

Merge branch 'main' into 11605

efdd37f

mergify bot merged commit c55367f into mozilla-mobile:main Jan 25, 2022

Provide feedback