Toggler does way too much and is impossible to reason about. Start by moving the mw.storage interactions to a new file, either a section storage specific file or a utility.
| Status | Subtype | Assigned | Task | ||
|---|---|---|---|---|---|
| Resolved | Jdlrobson | T221022 [EPIC] Divide MobileFrontend component concerns | |||
| Duplicate | None | T220850 Separate Toggler storage responsibilities |
We pointed this as a small because this task is just about isolating the storage and parse interactions. Toggler should only care about the un/marshaller not where it's stored. It's using JSON.parse / stringify which isn't great and could be better typed. Storage is already abstracted as mw.storage.