[superset] branch master updated: fix(explore): make to convert null to N/A in view results (#19316)

[rusackas]

This is an automated email from the ASF dual-hosted git repository.

rusackas pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/superset.git


The following commit(s) were added to refs/heads/master by this push:
     new 468c5ca  fix(explore): make to convert null to N/A in view results 
(#19316)
468c5ca is described below

commit 468c5ca29a42a1b602de75eb4a2f0aed70dfdf2e
Author: smileydev <47900232+prosdev0...@users.noreply.github.com>
AuthorDate: Wed Mar 23 10:02:52 2022 -0400

    fix(explore): make to convert null to N/A in view results (#19316)
    
    * fix(explore): make to convert null to N/A in view results
    
    * fix(explore): make to null formatter move before timeFormatter
---
 superset-frontend/src/constants.ts                                | 5 +++++
 .../src/explore/components/DataTableControl/index.tsx             | 8 ++++++++
 2 files changed, 13 insertions(+)

diff --git a/superset-frontend/src/constants.ts 
b/superset-frontend/src/constants.ts
index 2ef1bc7..8377c5b 100644
--- a/superset-frontend/src/constants.ts
+++ b/superset-frontend/src/constants.ts
@@ -102,3 +102,8 @@ export const FAST_DEBOUNCE = 250;
  * Slower debounce delay for inputs with expensive API calls.
  */
 export const SLOW_DEBOUNCE = 500;
+
+/**
+ * Display null as `N/A`
+ */
+export const NULL_DISPLAY = 'N/A';
diff --git 
a/superset-frontend/src/explore/components/DataTableControl/index.tsx 
b/superset-frontend/src/explore/components/DataTableControl/index.tsx
index d2befa7..c94c07c 100644
--- a/superset-frontend/src/explore/components/DataTableControl/index.tsx
+++ b/superset-frontend/src/explore/components/DataTableControl/index.tsx
@@ -36,6 +36,7 @@ import {
   BOOL_FALSE_DISPLAY,
   BOOL_TRUE_DISPLAY,
   SLOW_DEBOUNCE,
+  NULL_DISPLAY,
 } from 'src/constants';
 import { Radio } from 'src/components/Radio';
 import Icons from 'src/components/Icons';
@@ -49,6 +50,10 @@ import {
   unsetTimeFormattedColumn,
 } from 'src/explore/actions/exploreActions';
 
+export const CellNull = styled('span')`
+  color: ${({ theme }) => theme.colors.grayscale.light1};
+`;
+
 export const CopyButton = styled(Button)`
   font-size: ${({ theme }) => theme.typography.sizes.s}px;
 
@@ -303,6 +308,9 @@ export const useTableColumns = (
                   if (value === false) {
                     return BOOL_FALSE_DISPLAY;
                   }
+                  if (value === null) {
+                    return <CellNull>{NULL_DISPLAY}</CellNull>;
+                  }
                   if (timeFormattedColumnIndex > -1) {
                     return timeFormatter(value);
                   }