Change Title's inner value and message type to Text
Status
[x] Ready
[ ] Development
[ ] Hold
Description
This PR changes Title to use Text instead of json-serialized String and messages to use Text instead of raw String, adds a new quill->host serialization method (json). This change was made because bincode ^1 doesn't support deserialize_any but Text is an untagged enum and requires deserialize_any.
Related issues
Checklist
[x] Ran cargo fmt, cargo clippy --all-targets, cargo build --release and cargo test and fixed any generated errors!
[x] Removed unnecessary commented out code
[x] Used specific traces (if you trace actions please specify the cause i.e. the player)
Note: if you locally don't get any errors, but GitHub Actions fails (especially at clippy) you might want to check your rust toolchain version. You can then feel free to fix these warnings/errors in your PR.
Change Title's inner value and message type to
TextStatus
Description
This PR changes
Titleto useTextinstead of json-serializedStringand messages to useTextinstead of rawString, adds a new quill->host serialization method (json). This change was made because bincode ^1 doesn't supportdeserialize_anybutTextis an untagged enum and requiresdeserialize_any.Related issues
Checklist
cargo fmt,cargo clippy --all-targets,cargo build --releaseandcargo testand fixed any generated errors!Note: if you locally don't get any errors, but GitHub Actions fails (especially at
clippy) you might want to check your rust toolchain version. You can then feel free to fix these warnings/errors in your PR.