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

JS API returns undefined for null string values #1917

Open
Tracked by #5927
mattrunyon opened this issue Feb 1, 2022 · 0 comments
Open
Tracked by #5927

JS API returns undefined for null string values #1917

mattrunyon opened this issue Feb 1, 2022 · 0 comments
Assignees
Labels
Milestone

Comments

@mattrunyon
Copy link
Contributor

Description

The JS API gives undefined instead of null for null string values.

Steps to reproduce
Run this Python snippet

from deephaven.TableTools import newTable, stringCol, charCol

source = newTable(
    stringCol("Value", "A" , "B", "C", "D", "E", "", None),
    charCol("Char", "A", "B", "C", "D", "E", " ", None)
)

Expected results

The data created in the viewport has value set to null for the last row in both columns

Actual results

Only the char column value is null. String column value is undefined.

Additional details and attachments

If applicable, add any additional screenshots, logs, or other attachments to help explain your problem.

Versions

  • Deephaven: 0.8.0
  • Browser: Chrome
@mattrunyon mattrunyon added bug Something isn't working jsapi triage labels Feb 1, 2022
mofojed pushed a commit that referenced this issue Apr 4, 2024
Release notes https://github.com/deephaven/web-client-ui/releases/tag/v0.72.0

# [0.72.0](deephaven/web-client-ui@v0.71.0...v0.72.0) (2024-04-04)


### Bug Fixes

* Add isInvalid prop to Select component ([#1883](deephaven/web-client-ui#1883)) ([1803f31](deephaven/web-client-ui@1803f31)), closes [#1882](deephaven/web-client-ui#1882)
* adjust alignment of search input next/previous buttons ([#1917](deephaven/web-client-ui#1917)) ([c7fcd38](deephaven/web-client-ui@c7fcd38))
* Dashboard onLayoutInitialized not firing if config is empty ([#1914](deephaven/web-client-ui#1914)) ([84c648b](deephaven/web-client-ui@84c648b))
* package-lock missing pinned dependency ([#1920](deephaven/web-client-ui#1920)) ([1d9d216](deephaven/web-client-ui@1d9d216))


### Code Refactoring

* Change embed-grid and embed-chart to redirects ([#1873](deephaven/web-client-ui#1873)) ([e17619a](deephaven/web-client-ui@e17619a))


### Features

* Picker - formatter settings ([#1907](deephaven/web-client-ui#1907)) ([f06a141](deephaven/web-client-ui@f06a141)), closes [#1889](deephaven/web-client-ui#1889)
* re-export spectrum useStyleProp util ([#1916](deephaven/web-client-ui#1916)) ([aafa14b](deephaven/web-client-ui@aafa14b))
* wrap spectrum View, Text and Heading to accept custom colors ([#1903](deephaven/web-client-ui#1903)) ([a03fa07](deephaven/web-client-ui@a03fa07))


### BREAKING CHANGES

* `@deephaven/embed-grid` does not handle messages to the
iframe for filtering or sorting the grid any more





Co-authored-by: deephaven-internal <66694643+deephaven-internal@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants