Return KVT from create_history_stream() #47

glyph · 2022-06-29T12:55:57Z

glyph commented

2022-06-29 12:55:57 +00:00

While working on lykin I realised that create_history_stream() was returning messages in the form of SsbMessageValue and not SsbMessageKVT. Returning a KVT provides more flexibility and information; most importantly, it includes the message key.

This PR updates the create_history_stream() method to return a stream of SsbMessageKVT.

I have bumped the minor version (which acts as the major version in our case, since we are still at 0.x.y) to reflect the breaking change of the API.

While working on [lykin](https://git.coopcloud.tech/glyph/lykin) I realised that `create_history_stream()` was returning messages in the form of `SsbMessageValue` and not `SsbMessageKVT`. Returning a KVT provides more flexibility and information; most importantly, it includes the message key. This PR updates the `create_history_stream()` method to return a stream of `SsbMessageKVT`. I have bumped the minor version (which acts as the major version in our case, since we are still at `0.x.y`) to reflect the breaking change of the API.

glyph added 2 commits 2022-06-29 12:55:58 +00:00

bb07839691 return kvt instead of value

2abe4b5e10 bump version to reflect api change

glyph requested review from notplants 2022-06-29 12:56:14 +00:00

glyph added 1 commit 2022-06-29 12:57:02 +00:00

914ffb70cc fix formatting

notplants commented

2022-06-29 13:19:50 +00:00

looks good, can always easily transform sequences of kvts to message values

👍 1

glyph merged commit bf7d574064 into main

2022-06-29 13:38:56 +00:00