Closed Bug 1565482 Opened 5 years ago Closed 3 years ago

[META] Centralize all client -> server communication in one place

Categories

(DevTools :: Console, task, P2)

task

Tracking

(Fission Milestone:Future)

RESOLVED FIXED
Fission Milestone Future

People

(Reporter: nchevobbe, Unassigned)

References

Details

(Keywords: meta, Whiteboard: dt-fission-future)

At the moment the console doesn't have a clearly defined pattern to handle the communication from the client to the server.

Sometimes we do it in actions, or in middleware/enhancers, sometimes directly from the component, sometimes in webconsole-ui, etc…

This is kinda getting in our way for the Fission project, as we need a clear view of all the client <-> server communications, to take appropriate architecture decisions in order to handle multiple processes/frames.

We should follow what's done in other panel (like netmonitor, debugger, …) and have everything in one place.
This would help us have a better view of all the possible communications.
This should also help us enforce what those communications can be at component level (i.e. a component shouldn't be able to simply grab the client and call any method it wants on it).

Let's have this as a meta so we can have smaller actionable bugs to work on

Summary: Centralize all client -> server communication in one place → [META] Centralize all client -> server communication in one place

Some links to the Network monitor panel code base for inspiration:

This object represents the single point used to fetch data or receive events from the backend

Connecting to Chrome backend over CDP

Honza

Blocks: dt-fission
Whiteboard: dt-fission
Depends on: 1567167
Depends on: 1567174
Depends on: 1568143
Depends on: 1570985
Depends on: 1571413

Tracking Fission DevTools bugs for Fission Nightly (M6)

Fission Milestone: --- → M6

dt-fission-reserve bugs do not need to block Fission Nightly (M6), but these bugs' summaries mention the word "Fission", so let's track them for Fission riding the trains to Beta (M7). We'll revisit these bugs before we ship Fission.

Fission Milestone: M6 → M7

Bulk change of all bugs with whiteboard tag of dt-fission to Fission MVP milestone.

Fission Milestone: M7 → MVP

Moving old "dt-fission" bugs to "dt-fission-future" because they don't block Fission MVP.

Fission Milestone: MVP → Future
Whiteboard: dt-fission → dt-fission-future

This can probably be done using the "commands" pattern we're discussing about in Bug 1691681

Depends on: 1691681

We can probably improve that even more, may be via bug 1568499, but it looks like the bulk of work has been completed here.

Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.