Skip to content

Revert "Fix - Viewport does not return to highlighted message after returning from thread"#70411

Merged
inimaga merged 1 commit intomainfrom
revert-67752-fix/61550
Sep 11, 2025
Merged

Revert "Fix - Viewport does not return to highlighted message after returning from thread"#70411
inimaga merged 1 commit intomainfrom
revert-67752-fix/61550

Conversation

@inimaga
Copy link
Contributor

@inimaga inimaga commented Sep 11, 2025

@inimaga inimaga requested a review from a team as a code owner September 11, 2025 15:45
@melvin-bot melvin-bot bot requested review from stitesExpensify and removed request for a team September 11, 2025 15:46
@melvin-bot
Copy link

melvin-bot bot commented Sep 11, 2025

@stitesExpensify 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]

@github-actions
Copy link
Contributor

🚧 @inimaga has triggered a test Expensify/App build. You can view the workflow run here.

@github-actions
Copy link
Contributor

🧪🧪 Use the links below to test this adhoc build on Android, iOS, Desktop, and Web. Happy testing! 🧪🧪
Built from App PR #70411.

Android 🤖 iOS 🍎
⏩ SKIPPED ⏩ ⏩ SKIPPED ⏩
The build for Android was skipped The build for iOS was skipped
Desktop 💻 Web 🕸️
⏩ SKIPPED ⏩ https://70411.pr-testing.expensify.com
The build for Desktop was skipped Web

👀 View the workflow run that generated this build 👀

@inimaga inimaga merged commit 27d7c34 into main Sep 11, 2025
24 of 28 checks passed
@inimaga inimaga deleted the revert-67752-fix/61550 branch September 11, 2025 16:23
@melvin-bot melvin-bot bot added the Emergency label Sep 11, 2025
@melvin-bot
Copy link

melvin-bot bot commented Sep 11, 2025

@inimaga looks like this was merged without a test passing. Please add a note explaining why this was done and remove the Emergency label if this is not an emergency.

@inimaga
Copy link
Contributor Author

inimaga commented Sep 11, 2025

Emergency not applicable as the failing test are PR author and reviewer checklist

@inimaga inimaga removed the Emergency label Sep 11, 2025
OSBotify pushed a commit that referenced this pull request Sep 11, 2025
Revert "Fix - Viewport does not return to highlighted message after returning from thread"

(cherry picked from commit 27d7c34)

(cherry-picked to staging by AndrewGable)
@OSBotify OSBotify added the CP Staging marks PRs that have been CP'd to staging label Sep 11, 2025
@OSBotify
Copy link
Contributor

✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

@OSBotify
Copy link
Contributor

🚀 Cherry-picked to staging by https://github.com/AndrewGable in version: 9.2.12-2 🚀

platform result
🖥 desktop 🖥 success ✅
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

@OSBotify
Copy link
Contributor

🚀 Deployed to production by https://github.com/francoisl in version: 9.2.12-4 🚀

platform result
🖥 desktop 🖥 success ✅
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 success ✅

@OSBotify
Copy link
Contributor

🚀 Cherry-picked to staging by https://github.com/AndrewGable in version: 9.2.13-0 🚀

platform result
🖥 desktop 🖥 success ✅
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

@OSBotify
Copy link
Contributor

🚀 Deployed to production by https://github.com/mountiny in version: 9.2.13-1 🚀

platform result
🖥 desktop 🖥 success ✅
🕸 web 🕸 success ✅
🤖 android 🤖 success ✅
🍎 iOS 🍎 success ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CP Staging marks PRs that have been CP'd to staging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants