1582870 - Reduxify console actions

Status: RESOLVED FIXED

(DevTools :: Console, task, P1)

Description

[Jason Laster [:jlast]]

Currently, the service container is the primary communication point for components and the context menus. It would be nice if most of the functionality lived in redux actions and actions could be the primary communication point.

The reasons for this are:

• redux actions have access to the panel, api (fronts/clients), and UI (hud, webConsoleUI)
• are more easily shared than a stateful object like service container
• more idiomatic in react apps

This proposal is a collection of ideas for starting the conversation:

• I added some redux action files (object, toolbox)
• I added a commands file for talking with the server (debuggerClient, ObjectClient, threadFront)
• I moved a bunch of functionality up into the UI and hud to de-prioritize the service container and make the thunk middleware work
• I moved the storeAsGlobal logic into redux so that it will be easier to fissionify it

I look forward to getting your thoughts. The mochitests should be passing, but I'm missing a bunch of context, so I'm sure things will change :)

Comment 1

[Jason Laster [:jlast]]

Attachment: Bug 1582870 - Reduxify console actions. r=nchevobbe

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/e433d94bc095
Reduxify console actions. r=nchevobbe

Comment 3

[Razvan Maries]

https://hg.mozilla.org/mozilla-central/rev/e433d94bc095