Merge pull request #4 from Overmuse/SR/try_not_sending_order_id

remove order id field
Overmuse · Jun 21, 2021 · f2dc3d8 · f2dc3d8
2 parents 3aaeced + 771f46c
commit f2dc3d8
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 3 deletions.
diff --git a/.gitignore b/.gitignore
@@ -1 +1,2 @@
 /target
+.env
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -1,6 +1,6 @@
 [package]
 name = "reporting"
-version = "0.1.4"
+version = "0.1.5"
 authors = ["Sebastian Rollen <[email protected]>"]
 edition = "2018"
 

diff --git a/src/types.rs b/src/types.rs
@@ -13,7 +13,6 @@ impl ToMetrics for Order {
                 "status",
                 serde_plain::to_string(&self.status).context("Failed to serialize order status")?,
             )
-            .add_key_value("id", &self.client_order_id)
             .add_key_value("ticker", &self.symbol)
             .add_key_value(
                 "side",