Add alternative multiselect proposal #223
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Potential successor for #95. Initial work by @raulk and @marten-seemann. Surfacing it here to start a discussion.
Goal
Achieve enough consensus to start more serious prototyping by next Wednesday, October 30.
Needs
Concerns
I'll let @Stebalien speak for himself, but in our conversation he mentioned that, while he's interested in this iteration, he'd like for multiselect to remain as composable as is possible. While having a compact, prefix-analyzable binary format like that described in #95 is nice, switching to protobuf does make life easier for development and wouldn't sacrifice too much. He's a bit worried that this represents an over simplification
Notes & Thoughts
multiselect/string
, for example, and then have a receiver reply withmultiselect/dynamic
, mentioning the same protocol string the sender sent and then assigning it a dynamic ID.Cole's Conclusion
I think we could pick up #95 and modify it with some of the revelations from this iteration. In particular:
multiselect/string
andmultiselect/dyanmic
to match theOffer
andUse
flow.