To allow for transitioning from serde::Serialize to SerializeForVersion, the latter currently has a blanket implementation for all T: serde::Serialize.
Once the transition is complete, we can and should remove this blanket implementation. This will prove that we no longer rely on serde::Serialization directly.
You may find that this results in compilation errors in some DTO implementations. This will likely be due to missing primitive implementations for our trait. Various string or integer types spring to mind as likely culprits. These should be added explicitly to the serialization impl as demonstrated here.
Requires #2047.
To allow for transitioning from
serde::SerializetoSerializeForVersion, the latter currently has a blanket implementation for allT: serde::Serialize.Once the transition is complete, we can and should remove this blanket implementation. This will prove that we no longer rely on
serde::Serializationdirectly.You may find that this results in compilation errors in some DTO implementations. This will likely be due to missing primitive implementations for our trait. Various string or integer types spring to mind as likely culprits. These should be added explicitly to the serialization impl as demonstrated here.