Skip to content
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.

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

[EDITORIAL] Reclassify ConsumedUnit as Dimension instead of Metric #506

Conversation

kk09v
Copy link
Contributor

@kk09v kk09v commented Jul 11, 2024

Correct ConsumedUnit column type to Dimension from Metric (similar to PricingUnit) because it is an attribute of how the consumption is measured and is not able to have mathematical operations applied to it.

Correct ConsumedUnit column type to Dimension from Metric because it is an attribute of how the consumption is measured and is not able to have mathematical operations applied to it.
@kk09v kk09v requested a review from a team as a code owner July 11, 2024 19:30
@kk09v kk09v added bug in version A bug has been identified in one of the existing versions editorial Items that address editorial and minor changes labels Jul 11, 2024
@kk09v kk09v added this to the v1.1 milestone Jul 11, 2024
@jpradocueva jpradocueva added the P1 label Jul 15, 2024
@jpradocueva
Copy link
Contributor

Approved by the Members' group on July 18.

@jpradocueva jpradocueva merged commit 369ae31 into FinOps-Open-Cost-and-Usage-Spec:working_draft Jul 18, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug in version A bug has been identified in one of the existing versions editorial Items that address editorial and minor changes
Projects
Status: W.I.P
Development

Successfully merging this pull request may close these issues.

5 participants