Skip to content

[CP Staging] Revert "Fix: User is dropped in the Inbox after enabling camera permission and replacing receipt" - #70979

Merged
mountiny merged 1 commit into
mainfrom
revert-67245-fix-65992
Sep 22, 2025
Merged

mountiny merged 1 commit into
mainfrom
revert-67245-fix-65992

Conversation

@mountiny

Copy link
Copy Markdown
Contributor

Reverts #67245

Straight revert

Tests/ QA

Pre-condition: As admin , add a member in workspace

  1. Launch app as member
  2. Open employee workspace chat
  3. Create a manual expense
  4. Open the expense
  5. Tap plus icon and add a receipt
  6. After adding receipt in employee workspace, user must be directed to expense details page.

@mountiny mountiny self-assigned this Sep 22, 2025
@mountiny
mountiny requested a review from a team as a code owner September 22, 2025 12:44
@melvin-bot
melvin-bot Bot requested review from inimaga and removed request for a team September 22, 2025 12:45
@melvin-bot

melvin-bot Bot commented Sep 22, 2025

Copy link
Copy Markdown

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

@mountiny
mountiny merged commit 8d8705c into main Sep 22, 2025
21 of 23 checks passed
@mountiny
mountiny deleted the revert-67245-fix-65992 branch September 22, 2025 12:46
@melvin-bot

melvin-bot Bot commented Sep 22, 2025

Copy link
Copy Markdown

@mountiny 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.

@mountiny

Copy link
Copy Markdown
Contributor Author

Straight revert

@OSBotify OSBotify added the CP Staging marks PRs that have been CP'd to staging label Sep 22, 2025
OSBotify pushed a commit that referenced this pull request Sep 22, 2025
[CP Staging] Revert "Fix: User is dropped in the Inbox after enabling camera permission and replacing receipt"

(cherry picked from commit 8d8705c)

(cherry-picked to staging by mountiny)
@OSBotify

Copy link
Copy Markdown
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
Copy Markdown
Contributor

🚀 Cherry-picked to staging by https://github.com/mountiny in version: 9.2.16-3 🚀

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
Copy Markdown
Contributor

🚀 Cherry-picked to staging by https://github.com/mountiny in version: 9.2.16-10 🚀

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
Copy Markdown
Contributor

🚀 Deployed to production by https://github.com/roryabraham in version: 9.2.16-14 🚀

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

@OSBotify

Copy link
Copy Markdown
Contributor

🚀 Cherry-picked to staging by https://github.com/mountiny in version: 9.2.17-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
Copy Markdown
Contributor

🚀 Deployed to production by https://github.com/roryabraham in version: 9.2.17-6 🚀

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