fix: Open custom safe apps on click

[usame-algan]

What it solves

Resolves #2349

How this PR fixes it

• Moves the check if a safe app was previously open further up the call tree into handleSafeAppClick

How to test it

1. Open the custom safe apps tab
2. Add a custom safe app
3. Click on it
4. Observe that it directly opens
5. Navigate to the safe apps list
6. Click on a safe app
7. Observe that the drawer opens
8. Open that safe app
9. Navigate back to the safe apps list
10. Click on the same safe app again
11. Observe that it opens directly without a drawer

Checklist

• I've tested the branch on mobile 📱
• I've documented how it affects the analytics (if at all) 📊
• I've written a unit/e2e test for it (if applicable) 🧑‍💻

[github-actions]

Branch preview

✅ Deploy successful!

https://fix_custom_safe_apps--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}

[katspaugh]

👍

[francovenica]

LGTM.
Custom apps now open fine, with no drawer. You can still open the drawer if you click the " i " icon
Regular apps still open the drawer the 1st time they are opened and not more after that

Note: The test failing seems to be the Gorli/Goerli thing. We should standarize that instead of using regex in the tests