fix: Added connections logic specific to dapp

[NidhiKJha]

This PR is to ensure that the correct connected accounts show up when you visit the connections page via the permissions page

Related issues

Fixes: MetaMask-planning/issues/2282

Manual testing steps

1. Go to permissions page
2. check the permission page redirect you to connection page correctly

Screenshots/Recordings

Before

Screen.Recording.2024-03-28.at.3.25.16.PM.mov

After

Screen.Recording.2024-04-09.at.6.46.46.PM.mov

Pre-merge author checklist

• I’ve followed MetaMask Coding Standards.
• I've completed the PR template to the best of my ability
• I’ve included tests if applicable
• I’ve documented my code using JSDoc format if applicable
• I’ve applied the right labels on the PR (see labeling guidelines). Not required for external contributors.

Pre-merge reviewer checklist

• I've manually tested the PR (e.g. pull and build branch, run the app, test code being changed).
• I confirm that this PR addresses all acceptance criteria described in the ticket it closes and includes the necessary testing evidence such as recordings and or screenshots.

[darkwing]

This doesn't appear to be working as expected -- no accounts are listed:

Screen.Recording.2024-04-09.at.5.01.38.PM.mov

[darkwing]

After this is fixed, we need to pick up #23789

[github-actions]

CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes.

[codecov]

Codecov Report

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

Project coverage is 67.54%. Comparing base (c0e3ca0) to head (52a13a2).

Files	Patch %	Lines
ui/selectors/permissions.js	43.33%	17 Missing ⚠️
ui/components/multichain/app-header/app-header.js	0.00%	1 Missing ⚠️
...tichain/pages/permissions-page/permissions-page.js	0.00%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop   #23844      +/-   ##
===========================================
+ Coverage    67.53%   67.54%   +0.01%     
===========================================
  Files         1245     1245              
  Lines        48848    48864      +16     
  Branches     12743    12744       +1     
===========================================
+ Hits         32989    33003      +14     
- Misses       15859    15861       +2     

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

[metamaskbot]

Builds ready [52a13a2]

• builds: chrome, firefox
• builds (beta): chrome
• builds (flask): chrome, firefox
• builds (MMI): chrome, firefox
• builds (test): chrome, firefox
• builds (test-flask): chrome, firefox
• build viz: Build System
• mv3: Background Module Init Stats
• mv3: UI Init Stats
• mv3: Module Load Stats
• mv3: Bundle Size Stats
• mv2: E2e Actions Stats
• code coverage: Report
• storybook: Storybook
• typescript migration: Dashboard
• all artifacts
• bundle viz:

  • background: 0, 1, 2, 3, 4, 5, 6
  • common: 0, 1, 2, 3, 4, 5, 6, 7, 8
  • content-script: 0
  • ui: 0, 1, 2, 3, 4, 5, 6, 7, 8, 9

Page Load Metrics (898 ± 539 ms)

Platform	Page	Metric	Min (ms)	Max (ms)	Average (ms)	StandardDeviation (ms)	MarginOfError (ms)
Chrome	Home	firstPaint	60	361	143	84	40
		domContentLoaded	9	94	29	19	9
		load	48	2991	898	1123	539
		domInteractive	9	94	29	19	9

Bundle size diffs [🚨 Warning! Bundle size has increased!]

• background: 0 Bytes (0.00%)
• ui: -211 Bytes (-0.00%)
• common: 543 Bytes (0.01%)

[jonybur]

Why do we retain this empty span?