fix(ECharts): Revert ECharts version bump

[geido]

SUMMARY

PR #29963 has bumped ECharts to version 5.5.1. However, we have seen several issues with running the frontend. This PR reverts ECharts to previous version 5.4.1.

BEFORE

TESTING INSTRUCTIONS

1. Open a Dashboard with ECharts
2. ECharts should load

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

[kgabryje]

I guess that's our culprit... (from changelog)

[michael-s-molina]

@rusackas @hainenber Maybe we should require PR authors to include the CHANGELOG in the PR description when bumping minor/major versions?

[michael-s-molina]

By the way, it seems ECharts does not respect semver as this was a minor bump with a breaking change given @kgabryje's screenshot. This will require additional attention when bumping ECharts.

@villebro for awareness.

[hainenber]

Apologies all for the mess I made and thanks for fixing the horror @geido 👏

@michael-s-molina I will be more dilligent in bumping minor versions in the future. Semver is only a convention, after all 🤒

[michael-s-molina]

Apologies all for the mess I made and thanks for fixing the horror @geido 👏

@michael-s-molina I will be more dilligent in bumping minor versions in the future. Semver is only a convention, after all 🤒

No problem at all @hainenber. We really appreciate all the amazing contributions you have made to Superset. My comment is just to keep improving the process 😉

[hainenber]

Thanks. It's this amazing Superset community that aspire me to improve the platform further! 🙌

I think we might want to have some sort of Echarts integration test to keep ourselves less paranoid when it comes to bumping its version.