Skip to content

Commit

Permalink
Use date range in DateSpec instead of singular dates
Browse files Browse the repository at this point in the history
  • Loading branch information
jcorilla committed Jun 5, 2023
1 parent 491e7fd commit f54f107
Showing 1 changed file with 10 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -83,13 +83,19 @@ message SyntheticEventGroupSpec {
repeated VidRangeSpec vid_range_specs = 2;
}

// The specification for all frequencies reached at a specific date.
// The specification for all frequencies reached for a specific date range.
message DateSpec {
// Date the VIDs were reached.
google.type.Date date = 1;
// A range of `Date`s represented with a beginning and exclusive end.
message DateRange {
google.type.Date start = 1;
google.type.Date end_exclusive = 2;
}
// Dates the VIDs were reached.
DateRange date_range = 1;

// Each FrequencySpec must have a unique frequency.
repeated FrequencySpec frequency_specs = 2;
}

// `DateSpec`s should describe non-overlapping date ranges.
repeated DateSpec date_specs = 2;
}

0 comments on commit f54f107

Please sign in to comment.