Register Datasource Picker in the top nav menu for Get Started Tab

[derek-ho]

Description

Registers the new DataSource picker component from core in the top nav menu for the "Get Started" Tab.

Category

[Enhancement, New feature, Bug fix, Test fix, Refactoring, Maintenance, Documentation]
New feature

Why these changes are required?

• Resolves [FEATURE] Support multiple datasources in the "Get Started" tab #1796
• Resolves [FEATURE] Generalize DataSourcePicker/datasourceid usage #1816
• Resolves [FEATURE] Add dependency to dataSourceManagement bundle #1795

What is the old behavior before changes and new behavior after changes?

Old behavior is no selector, new behavior is shown in this video:

Screen.Recording.2024-03-20.at.10.53.21.AM.mov

Issues Resolved

[List any issues this PR will resolve (Is this a backport? If so, please add backport PR # and/or commits #)]

Fix: #1796, Fix: #1816 Fix: #1795

Testing

Manual testing, unit tests added, e2e cypress test added

Check List

• New functionality includes testing
• New functionality has been documented
• Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

[codecov]

Codecov Report

Attention: Patch coverage is 73.68421% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 67.65%. Comparing base (942b2f7) to head (be2c0fb).

Files	Patch %	Lines
public/apps/configuration/utils/request-utils.ts	0.00%	2 Missing ⚠️
public/apps/configuration/configuration-app.tsx	0.00%	1 Missing ⚠️
public/apps/configuration/top-nav-menu.tsx	85.71%	1 Missing ⚠️
public/plugin.ts	0.00%	1 Missing ⚠️

Additional details and impacted files

@@                             Coverage Diff                             @@
##           feature/supporting-multiple-datasources    #1818      +/-   ##
===========================================================================
+ Coverage                                    67.52%   67.65%   +0.13%     
===========================================================================
  Files                                           94       95       +1     
  Lines                                         2414     2427      +13     
  Branches                                       327      330       +3     
===========================================================================
+ Hits                                          1630     1642      +12     
- Misses                                         706      707       +1     
  Partials                                        78       78              

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

[DarshitChanpura]

looks close to me from the first pass. left some comments. Thanks @derek-ho for taking this on!

[cwperks]

Should dataSourceId(s) always be passed in as query params?

[derek-ho]

Yea, since we control all the places this is called (only in the get started tab) I think it's fine to require it. This isn't using query params though it's only an internal implementation. However I think this is a two way door decision and once more routes are modified we can easily refactor it to a cleaner way if we find one

[derek-ho]

If that makes sense can you merge? Thanks!