refactor: remove redundant propTypes in ColumnElement

[EugeneTorap]

SUMMARY

Refactoring ColumnElement component and remove redundant propTypes code

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 #18144 (7576bb2) into master (0cec0c9) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master   #18144      +/-   ##
==========================================
- Coverage   65.95%   65.92%   -0.03%     
==========================================
  Files        1584     1587       +3     
  Lines       62046    62116      +70     
  Branches     6273     6287      +14     
==========================================
+ Hits        40920    40950      +30     
- Misses      19505    19544      +39     
- Partials     1621     1622       +1     

Flag	Coverage Δ
javascript	50.86% <100.00%> (-0.02%)	⬇️

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

Impacted Files	Coverage Δ
...tend/src/SqlLab/components/ColumnElement/index.tsx	93.33% <100.00%> (-0.42%)	⬇️
...rc/explore/components/controls/TextAreaControl.jsx	80.00% <0.00%> (-4.22%)	⬇️
...nd/src/explore/components/ExploreActionButtons.tsx	59.45% <0.00%> (-3.40%)	⬇️
...ews/CRUD/annotationlayers/AnnotationLayerModal.tsx	58.97% <0.00%> (-1.56%)	⬇️
...c/filters/components/Select/SelectFilterPlugin.tsx	68.42% <0.00%> (-0.60%)	⬇️
...et-frontend/src/components/TableSelector/index.tsx	62.65% <0.00%> (-0.32%)	⬇️
...c/views/CRUD/data/database/DatabaseModal/index.tsx	32.46% <0.00%> (-0.22%)	⬇️
superset-frontend/src/preamble.ts	0.00% <0.00%> (ø)
superset-frontend/src/views/App.tsx	0.00% <0.00%> (ø)
superset-frontend/src/setup/setupClient.ts	0.00% <0.00%> (ø)
... and 29 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 0cec0c9...7576bb2. Read the comment docs.

[geido]

Just started the CI. I'll keep an eye

[EugeneTorap]

Just started the CI. I'll keep an eye

Can also test 18141, 18142, 18143 PR?

[EugeneTorap]

@lyndsiWilliams @AAfghahi Can you merge the PR?

[lyndsiWilliams]

I've got just one question, otherwise this looks good 😁

[lyndsiWilliams]

@lyndsiWilliams @AAfghahi Can you merge the PR?

We can review and approve PRs, but we can't merge unless there's also an approval from a higher-level engineer.

[EugeneTorap]

@lyndsiWilliams @AAfghahi Can you merge the PR?

We can review and approve PRs, but we can't merge unless there's also an approval from a higher-level engineer.

@lyndsiWilliams Got it, can you review it again?

[lyndsiWilliams]

@lyndsiWilliams @AAfghahi Can you merge the PR?

We can review and approve PRs, but we can't merge unless there's also an approval from a higher-level engineer.

@lyndsiWilliams Got it, can you review it again?

The code looks good! Could you fill in the description of your PR?

[EugeneTorap]

@lyndsiWilliams @AAfghahi Can you merge the PR?

We can review and approve PRs, but we can't merge unless there's also an approval from a higher-level engineer.

@lyndsiWilliams Got it, can you review it again?

The code looks good! Could you fill in the description of your PR?

Sure, done