Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Set a null default value to frecencyConfig and providerConfig in TopSitesConfig #11693

Closed
gabrielluong opened this issue Feb 9, 2022 · 0 comments · Fixed by #11694
Closed

Set a null default value to frecencyConfig and providerConfig in TopSitesConfig #11693

gabrielluong opened this issue Feb 9, 2022 · 0 comments · Fixed by #11694
Assignees
@gabrielluong
Labels
<top-sites> Component: feature-top-sites
Milestone
99.0.0 🛎

Comments

@gabrielluong
Copy link
Member

gabrielluong commented Feb 9, 2022 •
edited by data-sync-user

We should provide reasonable default to frecencyConfig and providerConfig in TopSitesConfig since we don't have to force the consumers to always provide a value.

┆Issue is synchronized with this Jira Task
@gabrielluong gabrielluong added the <top-sites> Component: feature-top-sites label Feb 9, 2022
@gabrielluong gabrielluong added this to the 99.0.0 🛎 milestone Feb 9, 2022
@gabrielluong gabrielluong self-assigned this Feb 9, 2022
gabrielluong added a commit to gabrielluong/android-components that referenced this issue Feb 9, 2022
@gabrielluong
Issue mozilla-mobile#11693: Set a null default value to frecencyConfi…
994100f 
…g and providerConfig in TopSitesConfig
@gabrielluong gabrielluong mentioned this issue Feb 9, 2022
Merged
4 tasks
@mergify mergify bot closed this as completed in #11694 Feb 9, 2022
mergify bot pushed a commit that referenced this issue Feb 9, 2022
@gabrielluong @mergify
Issue #11693: Set a null default value to frecencyConfig and provider…
3af2a0b 
…Config in TopSitesConfig
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@gabrielluong gabrielluong

Labels
<top-sites> Component: feature-top-sites
Projects
None yet
Milestone
99.0.0 🛎
1 participant
@gabrielluong