Skip to content

Revert "Accidentally navigating to Inbox from Reports page breaks workflow context"#74811

Closed
Beamanator wants to merge 1 commit into
mainfrom
revert-73738-paulnjs-fix/73260
Closed

Revert "Accidentally navigating to Inbox from Reports page breaks workflow context"#74811
Beamanator wants to merge 1 commit into
mainfrom
revert-73738-paulnjs-fix/73260

Conversation

@Beamanator

Copy link
Copy Markdown
Contributor

Reverts #73738

To fix: #74805

@Beamanator Beamanator marked this pull request as ready for review November 11, 2025 14:40
@Beamanator Beamanator requested review from a team as code owners November 11, 2025 14:40
@melvin-bot melvin-bot Bot requested review from JmillsExpensify and removed request for a team November 11, 2025 14:41
@github-actions

Copy link
Copy Markdown
Contributor

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

@melvin-bot melvin-bot Bot requested review from JS00001 and removed request for a team November 11, 2025 14:41
@melvin-bot

melvin-bot Bot commented Nov 11, 2025

Copy link
Copy Markdown

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

@truph01

truph01 commented Nov 11, 2025

Copy link
Copy Markdown
Contributor

I can reproduce the bug in production as well, so I'm curious if it was introduced by my original PR.

@github-actions

Copy link
Copy Markdown
Contributor

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

Android 🤖 iOS 🍎
https://ad-hoc-expensify-cash.s3.amazonaws.com/android/74811/index.html ⏩ SKIPPED ⏩
Android The build for iOS was skipped
Desktop 💻 Web 🕸️
https://ad-hoc-expensify-cash.s3.amazonaws.com/desktop/74811/NewExpensify.dmg https://74811.pr-testing.expensify.com
Desktop Web

👀 View the workflow run that generated this build 👀

@truph01

truph01 commented Nov 11, 2025

Copy link
Copy Markdown
Contributor

Here is a video in production:

Screen.Recording.2025-11-11.at.22.13.49.mov

@Beamanator Beamanator marked this pull request as draft November 11, 2025 15:15
@Beamanator

Copy link
Copy Markdown
Contributor Author

Innnnteresting, converting to draft for now so we don't revert quite yet

@Beamanator

Copy link
Copy Markdown
Contributor Author

seems the blocker is reproducible in prod, we don't need to revert

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[$250] Reports - Nothing happens after clicking Mark as paid on chat RHP

3 participants