You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Feb 22, 2020. It is now read-only.
With the store implemented, we should be able to use the value from it throughout the application and thus we can eliminate most EventEmitter implementation, since if there are communication cross component, it's best to do it once and use it everywhere.
Current Behavior
At the moment, the following can be refactored to consume store state:
ImageSeries - TreeView EvenEmitter when selecting a new image
Possible Solution
Compose a list of EventEmitter
Compose a list of store state that are nice to have
Refactor the EventEmitter into dispatcher and observer with the store.
Checklist before submitting
I have confirmed this using the officially supported Docker Compose setup using the local.yml configuration and ensured that I built the containers again and they reflect the most recent version of the project at the HEAD commit on the master branch
I have searched through the other currently open issues and am confident this is not a duplicate of an existing bug
I provided a minimal code snippet or list of steps that reproduces the bug.
I provided screenshots where appropriate
I filled out all the relevant sections of this template
The text was updated successfully, but these errors were encountered:
Expected Behavior
With the store implemented, we should be able to use the value from it throughout the application and thus we can eliminate most EventEmitter implementation, since if there are communication cross component, it's best to do it once and use it everywhere.
Current Behavior
At the moment, the following can be refactored to consume store state:
Possible Solution
Checklist before submitting
local.yml
configuration and ensured that I built the containers again and they reflect the most recent version of the project at theHEAD
commit on themaster
branchThe text was updated successfully, but these errors were encountered: