chore: Upgrade [email protected]

[geido]

SUMMARY

Upgrades Antd to version 4.10.3, in order to use property maxTagCount="responsive" in the Select component enabling dropdowns to display tags responsively.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

N.A.

TESTING INSTRUCTIONS

N.A.

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 #22007 (38764d5) into master (cd1b379) will not change coverage.
The diff coverage is n/a.

❗ Current head 38764d5 differs from pull request most recent head b49818e. Consider uploading reports for the commit b49818e to get more accurate results

@@           Coverage Diff           @@
##           master   #22007   +/-   ##
=======================================
  Coverage   67.07%   67.07%           
=======================================
  Files        1815     1815           
  Lines       69575    69575           
  Branches     7486     7486           
=======================================
  Hits        46665    46665           
  Misses      20974    20974           
  Partials     1936     1936           

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

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

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

[michael-s-molina]

@geido Adding the code below in superset-ui-core/test/__mocks__/resizer-observer-polyfill.ts should fix the test error:

export default function ResizeObserver(callback: ObserveCallback) {
  return {
    disconnect() {
      allCallbacks.splice(allCallbacks.indexOf(callback), 1);
    },
    observe() {
      if (callback) {
        allCallbacks.push(callback);
      }
    },
    unobserve() { <-- ADD THIS
      if (callback) {
        allCallbacks.splice(allCallbacks.indexOf(callback), 1);
      }
    },
  };
}

[geido]

Nice catch @michael-s-molina. I don't know how I didn't see this before!

[michael-s-molina]

Code LGTM.

[codyml]

LGTM - clicked around home, Explore Dashboard view/edit, datasets, SQL Lab and everything looked fine.

[jinghua-qa]

LGTM, run automation and did not see any blocker