This repository has been archived by the owner on Nov 1, 2022. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 474
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
7856: FxA Account Manager refactor r=csadilek a=grigoryk ### State machine changes: - "profile" states have been removed, `profile` is now handled outside of the state machine - states and events have been split into two subtypes: external and internal. External states are the `AccountState`s (authenticated, not authenticated, auth problem, etc) and internal states are `InternalState` (beginning auth, completing auth, recovering from auth problems, etc) - this split should hopefully make navigating the state machine more intuitive - retry logic and error handling was added for all internal transitions which require networking (Closes #7536) ### account manager changes: - removed ability to "set sync config" from the public API. we weren't using it (and have no plans to use this), so the added complexity wasn't benefiting anything. ### async/suspend API changes: account manager, account and device constellation APIs are now `suspend`, which makes both internal code and consuming of these APIs nicer and more flexible. Actual semantics of what's going on didn't change. ### sync manager changes: introduced a delay to the periodic sync scheduling, so that we can properly manage what goes on during startup (Closes #7335) Tests are half-way being updating, so this is still a WIP. cc @csadilek @rfk @eoger @jonalmeida Co-authored-by: Grisha Kruglov <gkruglov@mozilla.com>
- Loading branch information
Showing
42 changed files
with
1,992 additions
and
2,118 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.