Skip to content
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.

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

remove message type from Emit, assume type from Announce #7

Open
nikhan opened this issue Jul 28, 2014 · 1 comment
Open

remove message type from Emit, assume type from Announce #7

nikhan opened this issue Jul 28, 2014 · 1 comment
Labels

Comments

@nikhan
Copy link
Contributor

nikhan commented Jul 28, 2014

I may not fully understand the use case, but if a Colony service announces a type it is producing, shouldn't Emit() always produce messages given by function Announce()? For example, if I declare my service is sending bees should I have to specify that each message of type bees

The exception here is if a service wants to produce different kinds of messages than what it emits, but I would say that affordance could be confusing.

I just used Emit() to send a message type different than what I announced and I am not sure that is desirable behavior.

@nikhan nikhan changed the title remove message type from Emit remove message type from Emit, assume type from Announce Jul 28, 2014
@mikedewar
Copy link
Collaborator

Main difficulty there is that a service can emit multiple types.

Announce could define those content types that are allowable and emit could
error of the content types haven't been announced.
On Jul 28, 2014 5:36 PM, "nik hanselmann" [email protected] wrote:

I may not fully understand the use case, but if a Colony service announces
a type it is producing, shouldn't Emit() always produces messages given
by function Announce() For example, if I declare my service is sending
bees should I have to specify that each message of type bees

The exception here is if a service wants to produce different kinds of
messages than what it emits, but I would say that affordance could be
confusing.

I just used Emit() to send a message type different than what I announced
and I am not sure that is desirable behavior.


Reply to this email directly or view it on GitHub
#7.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants