Sam/security check infinite

[samholmes]

CHANGELOG

Does this branch warrant an entry to the CHANGELOG?

• Yes
• No

Dependencies

none

Requirements

If you have made any visual changes to the GUI. Make sure you have:

• Tested on iOS device
• Tested on Android device
• Tested on small-screen device (iPod Touch)
• Tested on large-screen device (tablet)

---

Note

Enable tab-aware layout for bank routing details, remove notifications in KYC form, clarify hasNotifications prop default, and tweak ESLint config.

• Scenes:
  • src/components/scenes/RampBankRoutingDetailsScene.tsx: Use SceneWrapper with hasTabs to account for tab bar insets.
  • src/components/scenes/RampKycFormScene.tsx: Remove hasNotifications from SceneWrapper; keep scroll/hasTabs/avoidKeyboard.
• Common:
  • src/components/common/SceneWrapper.tsx: Clarify comment for hasNotifications prop (default false).
• Tooling:
  • eslint.config.mjs: Remove src/components/common/SceneWrapper.tsx from the TypeScript warnings files list.

^{Written by Cursor Bugbot for commit 40b1a5d. This will update automatically on new commits. Configure here.}

---

• To see the specific tasks where the Asana app for GitHub is being used, see below:
  • https://app.asana.com/0/0/1211763080845108

[Jon-edge]

Did you audit all scenes to ensure such a sweeping change causes no regression in notification behavior?

FWIW I do agree the default should be false, but we need visibility into the expected changes.

[samholmes]

@Jon-edge I've audited the two scenes that were impacted.