Skip to content

Revert "fix: pay button appears in invoice preview for sender" #62509

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from

Conversation

Julesssss
Copy link
Contributor

Reverts #61659 to test possible fix for #62504

Copy link
Contributor

🚧 @Julesssss has triggered a test app build. You can view the workflow run here.

Copy link
Contributor

🧪🧪 Use the links below to test this adhoc build on Android, iOS, Desktop, and Web. Happy testing! 🧪🧪

Android 🤖 iOS 🍎
Android 🤖🔄 iOS 🍎🔄
https://ad-hoc-expensify-cash.s3.amazonaws.com/android/62509-hybrid/index.html https://ad-hoc-expensify-cash.s3.amazonaws.com/ios/62509-hybrid/index.html
Android iOS
Desktop 💻 Web 🕸️
https://ad-hoc-expensify-cash.s3.amazonaws.com/desktop/62509/NewExpensify.dmg https://62509.pr-testing.expensify.com
Desktop Web

👀 View the workflow run that generated this build 👀

@Julesssss
Copy link
Contributor Author

It prevents the expanded report view from displaying. But I don't think that's the desired solution for this bug...

Before revert:
Screenshot 2025-05-21 at 14 01 01

After revert:
Screenshot 2025-05-21 at 13 45 52

@Julesssss
Copy link
Contributor Author

Pretty sure this PR was not the cause of the blocker. On the test build I can see the extended report and the button issue hasn't been resolved:

Screenshot 2025-05-21 at 14 20 04

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.

1 participant