Skip to content

Commit

Permalink
Fix: Support columns with dots (#26659) (#26993)
Browse files Browse the repository at this point in the history
Closes #26405

Upgrades tinymath so that columns with dots now work correctly.

---

To test, follow the details in #26405.

1. Get some data that has fields with dots in them. Beats data would work, or you can create a test index with just 2 documents as follows: 

```
POST test/test
{
  "string":"abc",
  "with.dot":"abc"
}
POST test/test
{
  "string":"abcd",
  "with.dot":"abcd"
}
```

2. Create a pie chart, splitting labels on one of the available fields.

Previously, given the POST info above, only `string` would work correctly. Now both (and both of their `.keyword` variants) work correctly.

![dec-04-2018 13-35-47](https://user-images.githubusercontent.com/404731/49473970-b2cf7a00-f7d0-11e8-995b-e1a5e2a2acba.gif)
  • Loading branch information
w33ble committed Dec 11, 2018
1 parent 56fe06f commit ebdba85
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion package.json
Expand Up @@ -233,7 +233,7 @@
"style-loader": "0.23.1",
"tar": "2.2.0",
"tinygradient": "0.3.0",
"tinymath": "1.1.0",
"tinymath": "1.1.1",
"topojson-client": "3.0.0",
"trunc-html": "1.0.2",
"trunc-text": "1.0.2",
Expand Down
2 changes: 1 addition & 1 deletion x-pack/package.json
Expand Up @@ -251,7 +251,7 @@
"styled-components": "3.3.3",
"tar-fs": "1.13.0",
"tinycolor2": "1.3.0",
"tinymath": "1.1.0",
"tinymath": "1.1.1",
"tslib": "^1.9.3",
"typescript-fsa": "^2.5.0",
"typescript-fsa-reducers": "^0.4.5",
Expand Down
8 changes: 4 additions & 4 deletions yarn.lock
Expand Up @@ -20039,10 +20039,10 @@ tinygradient@0.3.0:
dependencies:
tinycolor2 "~1.0.0"

tinymath@1.1.0:
version "1.1.0"
resolved "https://registry.yarnpkg.com/tinymath/-/tinymath-1.1.0.tgz#1d26f1c4c65d70f9c62563fa7cdab87d66e39eaa"
integrity sha512-v3PNQ6zGKAqlj4labyqeTzndakdc9ODccx8IgrX1f+yzPW9mY0nlhKTRAPYOjay8gdvTyEnl2bik7huffZ5f/g==
tinymath@1.1.1:
version "1.1.1"
resolved "https://registry.yarnpkg.com/tinymath/-/tinymath-1.1.1.tgz#67f42ee0bb351508fe3a9ce4cb8db019ac4500ca"
integrity sha512-dpnNUQvyO3AKvgJ5AtblOYuM/SJoSyjvhM7pstKzkVKUBRPMaSOi0+8h/QXMlfnOVFWV0qSpfxSAUAHKZ3XN2g==

title-case@^2.1.0:
version "2.1.1"
Expand Down

0 comments on commit ebdba85

Please sign in to comment.