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

correct volume unit #12744

Merged
merged 3 commits into from
Jul 30, 2018
Merged

correct volume unit #12744

merged 3 commits into from
Jul 30, 2018

Conversation

Worty
Copy link
Contributor

@Worty Worty commented Jul 27, 2018

changed 3 to ³

@CLAassistant
Copy link

CLAassistant commented Jul 27, 2018

CLA assistant check
All committers have signed the CLA.

Copy link
Contributor

@marefr marefr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See comment.

Would be nice if you could fix the Watt per square metre (W/m2) as well

{ text: 'milligram per normal cubic metre (mg/Nm3)', value: 'conmgNm3' },
{ text: 'gram per cubic metre (g/m3)', value: 'congm3' },
{ text: 'gram per normal cubic metre (g/Nm3)', value: 'congNm3' },
{ text: 'nanogram per normal cubic metre (ng/Nm³)', value: 'conngNm3' },
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You missed the one above nanogram per cubic metre (ng/m3)

@marefr marefr self-assigned this Jul 30, 2018
@marefr marefr merged commit 49430f7 into grafana:master Jul 30, 2018
@marefr
Copy link
Contributor

marefr commented Jul 30, 2018

Thank you for your first contribution to Grafana! Very appreciated!

@marefr marefr added this to the 5.3 milestone Jul 30, 2018
marefr added a commit that referenced this pull request Jul 30, 2018
ryantxu added a commit to NatelEnergy/grafana that referenced this pull request Aug 2, 2018
* grafana/master: (104 commits)
  Change to arrow functions
  Add all tests to one file
  changelog: add notes about closing grafana#12561
  Remove angularMocks
  All tests passing
  changelog: add notes about closing grafana#12762
  fix: team email tooltip was not showing
  fix: test data api route used old name for test data datasource, fixes grafana#12773
  changelog: add notes about closing grafana#12300
  Weird execution order for the tests...
  changelog: add notes about closing grafana#12744
  changelog: add notes about closing grafana#12727
  add aws_dx to cloudwatch datasource
  also fixed "Watt per square metre"
  fixed that missing one
  add version note to metric prefix and fix typo
  devenv: update sql dashboards
  mssql: update tests
  fix usage of metric column types so that you don't need to specify metric alias
  Begin conversion
  ...
ryantxu added a commit to ryantxu/grafana that referenced this pull request Aug 2, 2018
* grafana/master: (104 commits)
  Change to arrow functions
  Add all tests to one file
  changelog: add notes about closing grafana#12561
  Remove angularMocks
  All tests passing
  changelog: add notes about closing grafana#12762
  fix: team email tooltip was not showing
  fix: test data api route used old name for test data datasource, fixes grafana#12773
  changelog: add notes about closing grafana#12300
  Weird execution order for the tests...
  changelog: add notes about closing grafana#12744
  changelog: add notes about closing grafana#12727
  add aws_dx to cloudwatch datasource
  also fixed "Watt per square metre"
  fixed that missing one
  add version note to metric prefix and fix typo
  devenv: update sql dashboards
  mssql: update tests
  fix usage of metric column types so that you don't need to specify metric alias
  Begin conversion
  ...
ryantxu added a commit to ryantxu/grafana that referenced this pull request Aug 2, 2018
* grafana/master: (104 commits)
  Change to arrow functions
  Add all tests to one file
  changelog: add notes about closing grafana#12561
  Remove angularMocks
  All tests passing
  changelog: add notes about closing grafana#12762
  fix: team email tooltip was not showing
  fix: test data api route used old name for test data datasource, fixes grafana#12773
  changelog: add notes about closing grafana#12300
  Weird execution order for the tests...
  changelog: add notes about closing grafana#12744
  changelog: add notes about closing grafana#12727
  add aws_dx to cloudwatch datasource
  also fixed "Watt per square metre"
  fixed that missing one
  add version note to metric prefix and fix typo
  devenv: update sql dashboards
  mssql: update tests
  fix usage of metric column types so that you don't need to specify metric alias
  Begin conversion
  ...
ryantxu added a commit to ryantxu/grafana that referenced this pull request Aug 2, 2018
* grafana/master: (185 commits)
  Change to arrow functions
  Add all tests to one file
  changelog: add notes about closing grafana#12561
  Remove angularMocks
  All tests passing
  changelog: add notes about closing grafana#12762
  fix: team email tooltip was not showing
  fix: test data api route used old name for test data datasource, fixes grafana#12773
  changelog: add notes about closing grafana#12300
  Weird execution order for the tests...
  changelog: add notes about closing grafana#12744
  changelog: add notes about closing grafana#12727
  add aws_dx to cloudwatch datasource
  also fixed "Watt per square metre"
  fixed that missing one
  add version note to metric prefix and fix typo
  devenv: update sql dashboards
  mssql: update tests
  fix usage of metric column types so that you don't need to specify metric alias
  Begin conversion
  ...
ryantxu added a commit to ryantxu/grafana that referenced this pull request Aug 2, 2018
* grafana/master: (263 commits)
  Change to arrow functions
  Add all tests to one file
  changelog: add notes about closing grafana#12561
  Remove angularMocks
  All tests passing
  changelog: add notes about closing grafana#12762
  fix: team email tooltip was not showing
  fix: test data api route used old name for test data datasource, fixes grafana#12773
  changelog: add notes about closing grafana#12300
  Weird execution order for the tests...
  changelog: add notes about closing grafana#12744
  changelog: add notes about closing grafana#12727
  add aws_dx to cloudwatch datasource
  also fixed "Watt per square metre"
  fixed that missing one
  add version note to metric prefix and fix typo
  devenv: update sql dashboards
  mssql: update tests
  fix usage of metric column types so that you don't need to specify metric alias
  Begin conversion
  ...
@marefr marefr modified the milestones: 5.3, 5.3.0-beta1 Sep 6, 2018
@ying-jeanne ying-jeanne added the pr/external This PR is from external contributor label Apr 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/external This PR is from external contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants