1082729 - Roll out StoreMixin use to more places

Status: RESOLVED INCOMPLETE

(Hello (Loop) :: Client, defect, P3)

Description

[Dan Mosedale (:dmosedale, :dmose)]

Right now, both the room store and loop stores are created in conversation:jsx, and then passed down through the AppControllerView.

Ideally, we'd like to push the store creation down at least into the AppControllerView, possibly even further.

There's been some speculation about how to do this, with the possibilities including in componentDidMount or, alternately, given the AppControllerView it's own store and adding an initialization action of some sort that would create and pass stores into the appropriate places.

Comment 1

[Mark Banner (:standard8)]

Having re-read this and following on from bug 1106852 having been implemented, I think we should re-summarise this to roll-out use of the StoreMixin created in bug 1106852 to everywhere in the code.

This would eliminate the need to pass down the store mixins through all the components. If initialisation is still an issue later on, then I think it'll surface again at some stage.

Comment 2

[Mark Banner (:standard8)]

Support for Hello/Loop has been discontinued.

https://support.mozilla.org/kb/hello-status

Hence closing the old bugs. Thank you for your support.