[Backport 2.x] [Fix] VisType switching persistence and selectively show warning

[opensearch-trigger-bot]

Backport 2db2c43 from #3715.

[codecov-commenter]

Codecov Report

Merging #3760 (46771fb) into 2.x (c731e7c) will decrease coverage by 0.07%.
The diff coverage is 59.57%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##              2.x    #3760      +/-   ##
==========================================
- Coverage   66.46%   66.39%   -0.07%     
==========================================
  Files        3209     3209              
  Lines       61610    61617       +7     
  Branches     9504     9512       +8     
==========================================
- Hits        40948    40912      -36     
- Misses      18386    18421      +35     
- Partials     2276     2284       +8     

Flag	Coverage Δ
Linux	66.39% <59.57%> (-0.01%)	⬇️
Windows	?

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

Impacted Files	Coverage Δ
...uilder/public/application/components/right_nav.tsx	8.69% <0.00%> (-2.42%)	⬇️
...plication/utils/state_management/shared_actions.ts	100.00% <ø> (ø)
...blic/application/utils/get_persisted_agg_params.ts	84.84% <84.84%> (ø)

... and 7 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[ashwin-pc]

Cypress tests for this will fail until the backport is merged in the functional testing repo. So not fail the other CI workflows that are being backported, that test will only be merged after this one.

Ref Functional test backport: opensearch-project/opensearch-dashboards-functional-test#601