Skip to content

Commit

Permalink
Merge pull request #1689 from openmeterio/fix/meter-api
Browse files Browse the repository at this point in the history
fix(api): meter's valueProperty is not required when aggregation type is COUNT
  • Loading branch information
sagikazarmark authored Oct 16, 2024
2 parents cc307ae + 1fa7877 commit 908bf36
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 0 additions & 1 deletion api/openapi.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6497,7 +6497,6 @@ components:
- aggregation
- windowSize
- eventType
- valueProperty
properties:
id:
type: string
Expand Down
2 changes: 1 addition & 1 deletion api/spec/src/meters.tsp
Original file line number Diff line number Diff line change
Expand Up @@ -135,7 +135,7 @@ model Meter {
*/
@minLength(1)
@example("$.tokens")
valueProperty: string;
valueProperty?: string;

/**
* Named JSONPath expressions to extract the group by values from the event data.
Expand Down

0 comments on commit 908bf36

Please sign in to comment.