feat(explore): Implement data table empty states

[kgabryje]

SUMMARY

This PR implements chart empty states for 3 scenarios:

1. Not all required controls have values
2. Query returned no results
3. No samples were retrieved for dataset

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

1. Create a chart
2. Run a query that returns 0 results (for example set time range for last 5 minutes)
3. Remove required controls

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

CC @kasiazjc

https://app.shortcut.com/preset/story/36403/explore-empty-states

[codecov]

Codecov Report

Merging #18679 (5e09e0c) into master (a7d505d) will increase coverage by 0.08%.
The diff coverage is 68.42%.

❗ Current head 5e09e0c differs from pull request most recent head b0b80ab. Consider uploading reports for the commit b0b80ab to get more accurate results

@@            Coverage Diff             @@
##           master   #18679      +/-   ##
==========================================
+ Coverage   66.25%   66.34%   +0.08%     
==========================================
  Files        1604     1619      +15     
  Lines       62807    62941     +134     
  Branches     6333     6343      +10     
==========================================
+ Hits        41613    41756     +143     
+ Misses      19542    19533       -9     
  Partials     1652     1652              

Flag	Coverage Δ
javascript	51.32% <68.42%> (+0.11%)	⬆️

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

Impacted Files	Coverage Δ
superset-frontend/src/embedded/index.tsx	0.00% <0.00%> (ø)
...nd/src/explore/components/DataTablesPane/index.tsx	73.68% <0.00%> (-1.59%)	⬇️
...rontend/src/visualizations/TimeTable/TimeTable.jsx	0.00% <0.00%> (ø)
superset/db_engine_specs/trino.py	70.79% <ø> (+0.26%)	⬆️
superset/utils/pandas_postprocessing/__init__.py	100.00% <ø> (ø)
superset/utils/pandas_postprocessing/aggregate.py	90.90% <ø> (ø)
superset/utils/pandas_postprocessing/boxplot.py	100.00% <ø> (ø)
superset/utils/pandas_postprocessing/compare.py	88.88% <ø> (ø)
...perset/utils/pandas_postprocessing/contribution.py	100.00% <ø> (ø)
superset/utils/pandas_postprocessing/cum.py	100.00% <ø> (ø)
... and 12 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a7d505d...b0b80ab. Read the comment docs.

[villebro]

Two translation comments, other than that LGTM

[villebro]

I think we need to translate this like this:

Suggested change

	title={t(`No ${type} were returned for this query`)}
	title={t(`No %s were returned for this query`, type)}

[villebro]

Actually, we probably need to translate these like this to make sure the translators have the necessary context to fully translate the type strings:

if (type === 'sample') {
  return t(`No samples were returned for this query`)
}
if (type === 'result') {
  return t(`No results were returned for this query`)
}

I think this getting this right is pretty important, as these will be very visible fixtures in the application.

[kgabryje]

Done

[villebro]

Same here

[kgabryje]

Done