-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Hybrid Android - Device navigation bar is black and not green #55217
Comments
Triggered auto assignment to @twisterdotcom ( |
Triggered auto assignment to @srikarparsi ( |
💬 A slack conversation has been started in #expensify-open-source |
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
|
This is not a blocker, its because we turned of the edge to edge mode to fix the status overlapping issue. cc @WoLewicki @staszekscp would anyone from SWM want to work on enabling the edge to edge again? then I could assign them to the follow ups |
Our prod build from Play market is on 83-5, not repro on it. So I mark an issue as deploy screen-20250114-173048.1.mp4 |
Does this issue have very high priority? I think we could start working on it tomorrow. Also, we're working on migration Android app from |
I think if you can focus on the fragment tPR to have it ready this week or early next week, then it's fine. |
Should we assign @mateuuszzzzz now? |
How's it going @mateuuszzzzz? |
Hi @twisterdotcom, we changed PRs status to ready for a review. Those are complex PRs so QA process should be done before merging it to main OldDot: https://github.com/Expensify/Mobile-Expensify/pull/13349 |
QA is going to run tests on the test build |
Hi, we fixed 2 issues found during QA so I presume that we will be able to merge PR soon. It depends on the next round of QA, which should hopefully be a formality at this point 😅 |
@twisterdotcom @srikarparsi @mateuuszzzzz this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks! |
Hopefully, we will merge those PRs today. We passed QA last week and PRs are ready for a merge |
It was merged and deployed to staging |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Version Number: 9.0.85-0
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: No, reproducible on hybrid only
If this was caught during regression testing, add the test name, ID and link from TestRail: Exp
Email or phone of affected tester (no customers): Samsung Galaxy Z Fold 4 / Android 14
Issue reported by: Applause Internal Team
Device used: [email protected]
App Component: Left Hand Navigation (LHN)
Action Performed:
Precondition:
Expected Result:
Device navigation bar will flow app color, which is green.
Actual Result:
Device navigation bar is black.
This issue only happens in hybrid app.
Workaround:
Unknown
Platforms:
Screenshots/Videos
Bug6714002_1736864643876.Screen_Recording_20250114_222121_Expensify.1.mp4
Bug6714002_1736864612144!log.txt
View all open jobs on GitHub
Issue Owner
Current Issue Owner: @mateuuszzzzzThe text was updated successfully, but these errors were encountered: