fix: Display table data when there are periods in string column names

hydrosquall commented 3 years ago

Motivation

A redo of #41 (reverted b/c of integration test) + a test of whether NPM canary deploys via auto are working as intended. Patches adds an extra line to address the integration test that was detected in #55.
Looked at the Node integration test, realized that a column ID needs to be provided when we use function instead of string accessors (see logs: https://github.com/nteract/data-explorer/runs/2935372040 ).

Testing

This is the same change as #41, with an extra line to fix the integration test issue mentioned previously. A preview deploy of the library can be tested using the sample PR link indicated below.

📦 Published PR as canary version: 8.2.10--canary.64.d71fcbc.0

:sparkles: Test out this PR locally via: ```bash npm install @nteract/data-explorer@8.2.10--canary.64.d71fcbc.0 # or yarn add @nteract/data-explorer@8.2.10--canary.64.d71fcbc.0 ```

rgbkrk commented 3 years ago

Nice and clean.

github-actions[bot] commented 3 years ago

:rocket: PR was released in v8.2.10 :rocket:

nteract / data-explorer

fix: Display table data when there are periods in string column names #64

Motivation

Testing