Skip to content

Commit

Permalink
Do not lowercase dimension inputs while resolving
Browse files Browse the repository at this point in the history
  • Loading branch information
ayushjain17 committed Apr 25, 2024
1 parent 6c9eccf commit b8dea5b
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 3 deletions.
13 changes: 11 additions & 2 deletions crates/experimentation_platform/src/db/models.rs
Original file line number Diff line number Diff line change
@@ -1,12 +1,21 @@
use crate::db::schema::*;
use chrono::{DateTime, NaiveDateTime, Utc};

use diesel::{query_builder::QueryId, Insertable, Queryable, QueryableByName, Selectable};
use diesel::{
query_builder::QueryId, Insertable, Queryable, QueryableByName, Selectable,
};
use serde::{Deserialize, Serialize};
use serde_json::Value;

#[derive(
Debug, Clone, Copy, PartialEq, Deserialize, Serialize, diesel_derive_enum::DbEnum, QueryId,
Debug,
Clone,
Copy,
PartialEq,
Deserialize,
Serialize,
diesel_derive_enum::DbEnum,
QueryId,
)]
#[DbValueStyle = "UPPERCASE"]
#[ExistingTypePath = "crate::db::schema::sql_types::ExperimentStatusType"]
Expand Down
2 changes: 1 addition & 1 deletion crates/frontend/src/pages/Home/Home.rs
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ pub fn home() -> impl IntoView {
"==" => "=",
_ => break, // query params do not support the other operators : != and IN, do something differently later
};
context.push(format!("{}{op}{}", dimension, value.to_lowercase()));
context.push(format!("{}{op}{}", dimension, value));
}
context.join("&").to_string()
};
Expand Down

0 comments on commit b8dea5b

Please sign in to comment.