Set Legacy Discover Table as Default for Discover 2.0 Experience

[LDrago27]

Description

This PR aims at accomplishing following things

1. Remove the Enable the legacy Discover Table setting when we have the QueryEnhancement flag turned ON
2. Set the legacy Discover Table as default when QueryEnhancement flag is turned ON irrespective of users preference of Enable the legacy Discover toggle
3. When QueryEnhancement flag is turned OFF respect users preference of Enable the legacy Discover toggle while displaying the discover table

Issues Resolved

Screenshot

Meeting.Recording.-.Sahoo.Suchit.Instant.Meeting.7.mp4

Testing the changes

• Follow the steps shown in the above video to validate various scenarios listed above

Changelog

• skip

Check List

• All tests pass
  • yarn test:jest
  • yarn test:jest_integration
• New functionality includes testing.
• New functionality has been documented.
• Update CHANGELOG.md
• Commits are signed per the DCO using --signoff

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 3 lines in your changes missing coverage. Please review.

Project coverage is 64.05%. Comparing base (6ebc1e3) to head (9c0010c).
Report is 3 commits behind head on main.

Files with missing lines	Patch %	Lines
...plication/components/data_grid/data_grid_table.tsx	0.00%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #8233      +/-   ##
==========================================
- Coverage   64.05%   64.05%   -0.01%     
==========================================
  Files        3741     3741              
  Lines       88682    88683       +1     
  Branches    13817    13817              
==========================================
  Hits        56809    56809              
- Misses      31261    31262       +1     
  Partials      612      612              

Flag	Coverage Δ
Linux_1	30.05% <0.00%> (-0.01%)	⬇️
Linux_2	58.85% <ø> (ø)
Linux_3	40.34% <0.00%> (-0.01%)	⬇️
Linux_4	31.43% <ø> (ø)
Windows_1	30.06% <0.00%> (-0.01%)	⬇️
Windows_2	58.80% <ø> (ø)
Windows_3	40.34% <0.00%> (-0.01%)	⬇️
Windows_4	31.43% <ø> (ø)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ashwin-pc]

Actually i think we should remove this toggle all together in the new discover and always use the legacy table in Discover 2.0. Since we plan on revisiting the table differently in Discover 2.0, there should be no reason to show this switch until we are ready to implement that table.

Expected behaviour:

• On switchign to Discover 2.0 always show the legacy table
• on switching to the old Discover, show the table variant that the user had selected

We dont need this toggle in Discover 2.0

[LDrago27]

Actually i think we should remove this toggle all together in the new discover and always use the legacy table in Discover 2.0. Since we plan on revisiting the table differently in Discover 2.0, there should be no reason to show this switch until we are ready to implement that table.

Expected behaviour:

• On switchign to Discover 2.0 always show the legacy table
• on switching to the old Discover, show the table variant that the user had selected

@ashwin-pc Updated the PR and description as per the new requirements

[ashwin-pc]

Nice!