fix(big number): time grain control is useless in BigNumber Viz

[zhaoyongjie]

SUMMARY

The Time Grain control is useless in BigNumber Viz(without trendline).

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Before

After

TESTING INSTRUCTIONS

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

[codecov]

Codecov Report

Merging #21372 (48d95ce) into master (b302818) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #21372   +/-   ##
=======================================
  Coverage   66.61%   66.61%           
=======================================
  Files        1791     1791           
  Lines       68538    68538           
  Branches     7310     7310           
=======================================
  Hits        45655    45655           
  Misses      21002    21002           
  Partials     1881     1881           

Flag	Coverage Δ
javascript	52.79% <ø> (ø)

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

Impacted Files	Coverage Δ
...harts/src/BigNumber/BigNumberTotal/controlPanel.ts	100.00% <ø> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[john-bodley]

@zhaoyongjie should this change also contain a database migration which removes the time_grain_sqla parameters if the viz_type is big_number?

[zhaoyongjie]

@john-bodley the Big Number chart hasn't a columns-like control, so the time_grain never be applied on the chart. IMO, the database migration is unnecessary.