slack-rs / slack-rs-api

Rust interface for the Slack Web API
Apache License 2.0
109 stars 66 forks source link

timestamp precision fixes #92

Closed dten closed 4 years ago

dten commented 4 years ago

as was speculated many moons ago by @saethlin f64 did not have enough precision slack seems to use a string if f64 isn't enough precision, so in the string parsing case only parse the decimal part as an f64 before scaling it up and adding to the rest as u64

includes tests now!

also removes Timestamp into f64 as it's not safe but adds Display / to_string