Fix: prevent an early redirect on the Safe Apps share page

[katspaugh]

What it solves

Resolves #2320

How this PR fixes it

It should wait for the client-side router to kick in before attempting a redirect.

How to test

Open this link:

https://fix_app_share--walletweb.review-wallet-web.5afe.dev/share/safe-app?appUrl=https%3A%2F%2Fsafe-apps.dev.5afe.dev%2Fdrain-safe&chain=gor

It should not redirect to the Dashboard.

[github-actions]

Branch preview

✅ Deploy successful!

https://fix_app_share--walletweb.review-wallet-web.5afe.dev

[github-actions]

ESLint Summary View Full Report

Annotations are provided inline on the Files Changed tab. You can also see all annotations that were generated on the annotations page.

Type	Occurrences	Fixable
Errors	0	0
Warnings	0	0
Ignored	0	N/A

• Result: ✅ success
• Annotations: 0 total

---

^{Report generated by eslint-plus-action}

[iamacook]

!appUrl || !validChain || !chain can be moved to a variable as it's used twice.

[katspaugh]

Good idea but TypeScript no likey.

[francovenica]

LGTM.
I had to use a safari emulator to reproduce the issue in ther first place since as you said chrome (and FF) didn't show the issue.
The url provided works fine.