-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
e2e: fix build issue #45997
e2e: fix build issue #45997
Conversation
@Julesssss may I kindly ask you to have a look on this since you made a similar fix? cc @hannojg |
Ah maybe lets add the same comment that this is a temporary workaround that should be removed once X is fixed? |
@hannojg I think this comment is removed in |
Oh yeah, rather than leave TODO's we created this issue. If you could just comment on it that would be helpful. Thanks |
Ignoring checklist and E2E workflow failure, both are expected. |
@Julesssss looks like this was merged without a test passing. Please add a note explaining why this was done and remove the |
|
@Julesssss Looks like I don't have an access?
Ignore me, just saw your comment after page refresh. |
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
Oh of course. Don't worry in that case, I'll just add a note to that issue to remind us to update this workflow in addition to the others once the Fastlane fix is live. |
🚀 Deployed to staging by https://github.com/Julesssss in version: 9.0.12-0 🚀
|
🚀 Deployed to production by https://github.com/francoisl in version: 9.0.12-0 🚀
|
🚀 Deployed to staging by https://github.com/Julesssss in version: 9.0.13-0 🚀
|
Details
Added a workaround described in fastlane/fastlane#21944
We already use this workaround in Expensify codebase #45332
This PR just replicates it to e2e pipeline: https://github.com/Expensify/App/actions/runs/10055336146/job/27792333690
Fixed Issues
$ N/A
PROPOSAL: https://expensify.slack.com/archives/C06JLFDLLUX/p1721728641449399?thread_ts=1721728324.572059&cid=C06JLFDLLUX
Tests
Offline tests
N/A
QA Steps
N/A
PR Author Checklist
### Fixed Issues
section aboveTests
sectionOffline steps
sectionQA steps
sectiontoggleReport
and notonIconClick
)myBool && <MyComponent />
.src/languages/*
files and using the translation methodSTYLE.md
) were followedAvatar
, I verified the components usingAvatar
are working as expected)StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)
)Avatar
is modified, I verified thatAvatar
is working as expected in all cases)Design
label and/or tagged@Expensify/design
so the design team can review the changes.ScrollView
component to make it scrollable when more elements are added to the page.main
branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTest
steps.Screenshots/Videos
Android: Native
Android: mWeb Chrome
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari
MacOS: Desktop