Skip to content
This repository has been archived by the owner on Dec 10, 2021. It is now read-only.

Allow metrics arrays in form data #61

Merged
merged 1 commit into from
Dec 12, 2018

Conversation

xtinec
Copy link
Contributor

@xtinec xtinec commented Dec 12, 2018

馃彔 Internal

Metrics can take shape of an array of metrics in form data for a given metrics key. For example, { metrics: ['sum__num'] }. This PR changes Metrics class to handle the array case.

Metrics can take shape of an array of metrics in form data for a given metrics key. For example, `{ metrics: ['sum__num'] }`. This PR changes Metrics class to handle the array case.
@xtinec xtinec requested a review from a team December 12, 2018 00:34
@xtinec xtinec added the WIP Work in progress label Dec 12, 2018
@codecov
Copy link

codecov bot commented Dec 12, 2018

Codecov Report

Merging #61 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##           master    #61   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          65     65           
  Lines         625    629    +4     
  Branches       68     70    +2     
=====================================
+ Hits          625    629    +4
Impacted Files Coverage 螖
packages/superset-ui-chart/src/query/FormData.ts 100% <酶> (酶) 猬嗭笍
packages/superset-ui-chart/src/query/Metric.ts 100% <100%> (酶) 猬嗭笍

Continue to review full report at Codecov.

Legend - Click here to learn more
螖 = absolute <relative> (impact), 酶 = not affected, ? = missing data
Powered by Codecov. Last update f5695f8...45eff8d. Read the comment docs.

@xtinec xtinec added reviewable Ready for review and removed WIP Work in progress labels Dec 12, 2018
@kristw kristw added the #enhancement New feature or request label Dec 12, 2018
@xtinec xtinec merged commit 2896242 into master Dec 12, 2018
@delete-merged-branch delete-merged-branch bot deleted the xtinec--support-metric-arrays-form-data branch December 12, 2018 01:48
@kristw kristw added this to the v0.8.0 milestone Feb 1, 2019
kristw added a commit that referenced this pull request Apr 17, 2020
* feat: update line chart thumbnail

* fix: lint
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
#enhancement New feature or request reviewable Ready for review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants