New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Support multi callbacks #6

Closed

kMutagene opened this issue Oct 14, 2020 · 1 comment

Labels

Area:Backend Type: Enhancement

Milestone

Contributor

kMutagene commented Oct 14, 2020

Callbacks should be a DU like this:

type Callback
| SingleCallback ...
| MultiCallback ...

this will make serialization harder but making multiple outputs possible is important.

the 'Output' property of the callback (which is basically its id) needs to be constructed from the multi output objects
this id should then automatically be used to register the callback, abstracting the argument away from the registerCallback function.

kMutagene mentioned this issue

Roadmap and discussion for beta release #4

Open

11 tasks

kMutagene added Area:Backend Type: Enhancement labels

kMutagene added this to the Beta milestone

Contributor Author

kMutagene commented Feb 3, 2021

Implemented via 66ba10d (dont know why the linking of this issue didn't work). You can now create multiCallbacks via Callback.multiOut

kMutagene closed this as completed

kMutagene added a commit that referenced this issue


          Refactor callback #10, add multi-output callbacks #6

66ba10d

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment