Skip to content

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

Closed
Julesssss wants to merge 1 commit into
mainfrom
revert-61659-fix/61315
Closed

Revert "fix: pay button appears in invoice preview for sender"#62509
Julesssss wants to merge 1 commit into
mainfrom
revert-61659-fix/61315

Conversation

@Julesssss

Copy link
Copy Markdown
Contributor

Reverts #61659 to test possible fix for #62504

@github-actions

Copy link
Copy Markdown
Contributor

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

@github-actions

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