Skip to content

Conversation

@daiyam
Copy link
Contributor

@daiyam daiyam commented Sep 13, 2018

This change allows to use YAML instead of JSON for defining charts.

```chart(yaml)
type: pie
data:
  labels:
  - Red
  - Blue
  - Yellow
  datasets:
  - data:
    - 300
    - 50
    - 100
    backgroundColor:
    - "#FF6384"
    - "#36A2EB"
    - "#FFCE56"
    hoverBackgroundColor:
    - "#FF6384"
    - "#36A2EB"
    - "#FFCE56"
```

screenshot

It's depending on the pull request #2394.

@ZeroX-DG ZeroX-DG added the awaiting review ❇️ Pull request is awaiting a review. label Sep 14, 2018
@Rokt33r Rokt33r added awaiting changes 🖊️ Pull request has been reviewed, but contributor needs to make changes. and removed awaiting review ❇️ Pull request is awaiting a review. labels Sep 30, 2018
@Rokt33r
Copy link
Member

Rokt33r commented Sep 30, 2018

Could you fix the conflicts?

@Rokt33r Rokt33r added awaiting review ❇️ Pull request is awaiting a review. and removed awaiting changes 🖊️ Pull request has been reviewed, but contributor needs to make changes. labels Oct 12, 2018
@Rokt33r Rokt33r self-requested a review October 12, 2018 07:11
@Rokt33r Rokt33r added next release (v0.11.12) and removed awaiting review ❇️ Pull request is awaiting a review. labels Nov 8, 2018
@Rokt33r Rokt33r merged commit 53aa142 into BoostIO:master Nov 8, 2018
@daiyam daiyam deleted the chart-yaml branch November 8, 2018 04:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants