Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Implement TimeStamp serde serialisation #99

Merged
merged 1 commit into from
Jul 11, 2018
Merged

Implement TimeStamp serde serialisation #99

merged 1 commit into from
Jul 11, 2018

Conversation

stefano-pogliani
Copy link
Contributor

The bson::TimeStamp type does not currently implement serde::Serialise.
As a result things like the example below fail to compile.

use bson::TimeStamp;

#[derive(Serialise)]
pub struct Foo {
  ts: TimeStamp,
}

This PR would implement Serialise for bson::TimeStamp so the above code compiles.

@zonyitoo zonyitoo merged commit 24352d0 into mongodb:master Jul 11, 2018
@zonyitoo zonyitoo self-assigned this Jul 11, 2018
@stefano-pogliani stefano-pogliani deleted the serialise-timestamp branch July 11, 2018 16:19
lrlna pushed a commit to lrlna/bson-rs that referenced this pull request Feb 27, 2019
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants