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

fix: Fix column data appearing incorrectly when multiplier null #1194

Merged
merged 2 commits into from
Apr 3, 2023

Conversation

mofojed
Copy link
Member

@mofojed mofojed commented Apr 3, 2023

  • Issue with Formatting values - Enterprise uses multiplier: null for the default settings, but in Community we only expected undefined or number.
  • Just handle when multiplier is null as well
  • Fixes Grid reporting incorrect values #1193
  • Tested by manually adding a formatter with multiplier: null in LocalWorkspaceStorage, e.g.:
        formatter: [
          {
            columnType: 'decimal',
            columnName: 'Bid',
            format: {
              label: '',
              type: 'type-global',
              formatString: '###,##0.00###',
              multiplier: null,
            },
          },
        ],

- Issue with Formatting values - Enterprise uses `multiplier: null` for the default settings, but in Community we only expected `undefined` or `number`.
- Just handle when `multiplier` is `null` as well
- Fixes deephaven#1193
- Tested by manually adding a formatter with `multiplier: null` in LocalWorkspaceStorage, e.g.:
```
        formatter: [
          {
            columnType: 'decimal',
            columnName: 'Bid',
            format: {
              label: '',
              type: 'type-global',
              formatString: '###,##0.00###',
              multiplier: null,
            },
          },
        ],
```
@mofojed mofojed added the bug Something isn't working label Apr 3, 2023
@mofojed mofojed requested a review from mattrunyon April 3, 2023 15:49
@mofojed mofojed self-assigned this Apr 3, 2023
@mofojed mofojed requested a review from vbabich April 3, 2023 15:49
vbabich
vbabich previously approved these changes Apr 3, 2023
@codecov
Copy link

codecov bot commented Apr 3, 2023

Codecov Report

Merging #1194 (9313c2d) into main (dfdf356) will decrease coverage by 0.01%.
The diff coverage is 10.00%.

@@            Coverage Diff             @@
##             main    #1194      +/-   ##
==========================================
- Coverage   44.20%   44.20%   -0.01%     
==========================================
  Files         448      448              
  Lines       33430    33437       +7     
  Branches     8401     8404       +3     
==========================================
+ Hits        14778    14781       +3     
- Misses      18602    18606       +4     
  Partials       50       50              
Flag Coverage Δ
unit 44.20% <10.00%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...kages/dashboard-core-plugins/src/ConsolePlugin.tsx 20.83% <0.00%> (-0.09%) ⬇️
...ashboard-core-plugins/src/panels/NotebookPanel.tsx 1.59% <0.00%> (-0.02%) ⬇️
...dashboard-core-plugins/src/events/NotebookEvent.ts 100.00% <100.00%> (ø)
...api-utils/src/formatters/DecimalColumnFormatter.ts 95.83% <100.00%> (+2.08%) ⬆️
...api-utils/src/formatters/IntegerColumnFormatter.ts 95.34% <100.00%> (+2.32%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@mofojed mofojed merged commit e22e68d into deephaven:main Apr 3, 2023
@mofojed mofojed deleted the 1193-incorrect-values branch April 3, 2023 16:08
@github-actions github-actions bot locked and limited conversation to collaborators Apr 3, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Grid reporting incorrect values
2 participants