[CP Staging] Fix iOS - Home - Concierge prompt box remains expanded after sending attachment - #99537
Conversation
|
@codex please review |
|
Codex Review: Didn't find any major issues. More of your lovely PRs please. Reviewed commit: ℹ️ About Codex in GitHubCodex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback". |
|
@ZhenjaHorbach Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button] |
Reviewer Checklist
Screenshots/VideosAndroid: HybridApp2026-08-26.11.52.20.movAndroid: mWeb Chrome2026-08-26.11.52.20.moviOS: HybridApp2026-08-26.11.49.04.moviOS: mWeb Safari2026-08-26.11.50.39.movMacOS: Chrome / Safari2026-08-26.11.46.42.mov |
|
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
…rge-prompt-box/fix-prompt-expanded-after-sending-attachment [CP Staging] Fix iOS - Home - Concierge prompt box remains expanded after sending attachment (cherry picked from commit dd4d85c) (cherry-picked to staging by grgia)
|
🚧 grgia has triggered a test Expensify/App build. You can view the workflow run here. |
|
🧪🧪 Use the links below to test this adhoc build on Android, iOS, and Web. Happy testing! 🧪🧪
|
|
🔁 Filed a Staging retest request for deploy blockers #99522 after this PR was cherry-picked to staging. |
|
🚀 Cherry-picked to staging by https://github.com/grgia in version: 9.4.61-3 🚀
|
|
🤖 No help site changes required. This PR is a purely technical bug fix — on iOS native, clearing the Concierge prompt's controlled It restores the expected/correct behavior of an existing UI element and introduces no new user-facing feature, workflow, setting, label, or terminology. Why no docs update is needed
Since no changes are required, I did not create a draft help site PR. |
|
🚀 Deployed to production by https://github.com/AndrewGable in version: 9.4.61-4 🚀
Bundle Size Analysis (Sentry): |
|
🚀 Cherry-picked to staging by https://github.com/grgia in version: 9.4.62-1 🚀
|
|
🤖 No help site changes required (re-confirmed). I already reviewed this PR — it's a purely technical iOS-native bug fix (forcing the Concierge prompt's native input to clear so the composer collapses after sending an attachment). It touches only |
|
🚀 Deployed to production by https://github.com/AndrewGable in version: 9.4.62-4 🚀
Bundle Size Analysis (Sentry): |
Explanation of Change
On iOS native, clearing the Concierge prompt's controlled
valuedidn't reset the native input, so the composer stayed expanded (multi-line height) after sending.ConciergePromptBoxnow holds anuseAnimatedRefon theComposerand callsforceClearInputviascheduleOnUIinclearInput, matching the pattern already used inComposerWithSuggestions.Fixed Issues
$ #99522
PROPOSAL: N/A
Tests
Offline tests
N/A
QA Steps
Same as tests
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectionAvatar, I verified the components usingAvatarare working as expected)StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))npm run compress-svg)Avataris modified, I verified thatAvataris working as expected in all cases)Designlabel and/or tagged@Expensify/designso the design team can review the changes.mainbranch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTeststeps.Screenshots/Videos
Android: Native
Screen.Recording.2026-08-26.at.10.58.09.mov
Android: mWeb Chrome
iOS: Native
Screen.Recording.2026-08-26.at.10.55.43.mov
iOS: mWeb Safari
MacOS: Chrome / Safari
Screen.Recording.2026-08-26.at.11.01.47.mov