refactor(ExploreCtasResultsButton): convert to functional component

[ChristopherCFleming]

SUMMARY

Updated from class component into functional component. Moved onClick below Visualize.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

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

[AAfghahi]

/testenv up

[github-actions]

@AAfghahi Ephemeral environment spinning up at http://34.219.47.255:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

[codecov]

Codecov Report

Merging #17939 (93d1ec2) into master (5da3c45) will increase coverage by 0.21%.
The diff coverage is 22.22%.

@@            Coverage Diff             @@
##           master   #17939      +/-   ##
==========================================
+ Coverage   66.34%   66.56%   +0.21%     
==========================================
  Files        1569     1573       +4     
  Lines       61687    61799     +112     
  Branches     6241     6294      +53     
==========================================
+ Hits        40927    41134     +207     
+ Misses      19162    19013     -149     
- Partials     1598     1652      +54     

Flag	Coverage Δ
javascript	51.39% <22.22%> (+0.47%)	⬆️

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

Impacted Files	Coverage Δ
...frontend/src/SqlLab/components/ResultSet/index.tsx	50.73% <ø> (ø)
...lLab/components/ExploreCtasResultsButton/index.jsx	30.76% <22.22%> (+19.00%)	⬆️
...ugins/plugin-chart-echarts/src/Timeseries/index.ts	25.00% <0.00%> (-25.00%)	⬇️
.../plugin-chart-echarts/src/Timeseries/Step/index.ts	25.00% <0.00%> (-25.00%)	⬇️
.../plugin-chart-echarts/src/Timeseries/Area/index.ts	33.33% <0.00%> (-16.67%)	⬇️
...-chart-echarts/src/Timeseries/Regular/Bar/index.ts	33.33% <0.00%> (-16.67%)	⬇️
...chart-echarts/src/Timeseries/Regular/Line/index.ts	33.33% <0.00%> (-16.67%)	⬇️
...rt-echarts/src/Timeseries/Regular/Scatter/index.ts	33.33% <0.00%> (-16.67%)	⬇️
...echarts/src/Timeseries/Regular/SmoothLine/index.ts	33.33% <0.00%> (-16.67%)	⬇️
...rontend/src/explore/components/EmbedCodeButton.jsx	66.66% <0.00%> (-12.29%)	⬇️
... and 93 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 5da3c45...93d1ec2. Read the comment docs.

[eschutho]

/testenv up

[eschutho]

@AAfghahi we can't test ctas on the ephemeral, can we?

[github-actions]

@eschutho Ephemeral environment spinning up at http://35.87.218.85:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

[AAfghahi]

@AAfghahi we can't test ctas on the ephemeral, can we?

I don't think we can, but we also couldn't enable it with a SQLite example db. I can pull it down and test in postgres.

[lyndsiWilliams]

Great work on this, thank you!

[eschutho]

looking great @ChristopherCFleming! Just a few small nits.

[eschutho]

/testenv up

[github-actions]

@eschutho Ephemeral environment spinning up at http://34.215.151.120:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

[AAfghahi]

tested this locally to see that it works:

Screen.Recording.2022-02-08.at.2.27.11.PM.mov

[eschutho]

Looks great. Thank you @ChristopherCFleming!

[github-actions]

Ephemeral environment shutdown and build artifacts deleted.

[ChristopherCFleming]

Thank you! Appreciate your time and guidance.