search

bitcoin-dev-project / sim-ln

Payment activity generator for the lightning network
MIT License
63 stars 28 forks source link

Fixing the serializer for ValueOrRange #188

Closed bjohnson5 closed 5 months ago

bjohnson5 commented 5 months ago

Using a more specific serialization process for the ValueOrRange type. This will allow the type to be correctly serialized and deserialized.

Close #187

bjohnson5 commented 5 months ago

@carlaKC @sr-gi Let me know what you think about this. I have been using simln in another Rust project and came across this issue when serializing my activity definitions.

sr-gi commented 5 months ago

Nice catch! We don't use the serializer internally it went completely unnoticed.

LGTM