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

Add support for Action Channels that support targeting messages in multiple channels. #14

Open
rrbrambley opened this issue Jan 4, 2014 · 1 comment
Assignees

Comments

@rrbrambley
Copy link
Owner

Since message ids are unique across channels, there isn't technically a reason why a single action channel cannot be used for multiple target channels.

We could either:

  1. Not require a channel_id in the com.alwaysallthetime.action.metadata channel annotation, or
  2. Have an alternate key – channel_ids that is an array of channel ids to be used with the action channel.
@ghost ghost assigned rrbrambley Jan 4, 2014
@rrbrambley
Copy link
Owner Author

Still on the fence about this. Removed the notion of 'target channel' in this branch: https://github.com/rrbrambley/MessageBeast-Android/tree/enhancements/no-target-channel

I think there might be some value in keeping the 'channel_id' key in the metadata annotation because if you look at an action channel independently, it allows you to see which Channel the messages correspond to.

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

No branches or pull requests

1 participant