Skip to content

fix(check data model): Last reported_at may be null #8

New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Open
Sonlis wants to merge 2 commits into
base: master
Choose a base branch
from
Open

fix(check data model): Last reported_at may be null #8

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
e544407
fix(check data model): Last reported_at may be null
Sonlis Feb 22, 2026
73be26e
fix: country code / name is optional
Apr 16, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions src/lib.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -204,7 +204,7 @@ pub struct Check {
#[serde(rename = "numDistinctUsers")]
pub num_distinct_users: u64,
#[serde(rename = "lastReportedAt")]
pub last_reported_at: DateTime<Utc>,
pub last_reported_at: Option<DateTime<Utc>>,
pub reports: Option<Vec<CheckReport>>,
}

Expand All @@ -217,9 +217,9 @@ pub struct CheckReport {
#[serde(rename = "reporterId")]
pub reporter_id: u64,
#[serde(rename = "reporterCountryCode")]
pub reporter_country_code: String,
pub reporter_country_code: Option<String>,
#[serde(rename = "reporterCountryName")]
pub reporter_country_name: String,
pub reporter_country_name: Option<String>,
}

// Verbose flag if `false`, will exclude reports and the country name field.
Expand Down