-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
[$125] Split bill - Split Bill description on Details page is empty in Group chat #29496
Comments
Triggered auto assignment to @maddylewis ( |
Bug0 Triage Checklist (Main S/O)
|
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open
|
Triggered auto assignment to @bondydaa ( |
discussed here, removing the blocker and exporting this |
Job added to Upwork: https://www.upwork.com/jobs/~01b2dd27f3e4c1c58d |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @situchan ( |
Upwork job price has been updated to $125 |
Regression from #29055 |
@jeet-dhandha can you look into this regression from your PR please? |
@bernhardoj Please share a video with the PR’s code and a video after reverting the code. As then only i can be sure its a regression from my PR. And also do check that the bugs fixed in the PR are also not coming back.if they are then the regression loop will go on forever. Thanks in advance. |
I don't have my laptop with me right now, but before #29055 you made a platform specific file for the MenuItem RenderHTML where the native file doesn't have the View wrapper. App/src/components/MenuItem.js Lines 255 to 257 in 2b4e1c6
You also did explain here that wrapping the RenderHTML with View on native will make the text empty. |
@bondydaa, @maddylewis, @situchan Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
@bernhardoj @jeet-dhandha what's the status here? |
I am OOO this and next week due to health reasons. |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
@situchan - have you been able to reproduce this bug recently? |
I am not able to reproduce on latest main |
Confirming you tested on Android native? |
Tested android as well. |
going to close this out then |
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: 1.3.83-5
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Expected Result:
Description should be saved and visible
Actual Result:
Description field is empty
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Android: Native
Bug6234948_1697141202004.az_recorder_20231012_204939.mp4
Android: mWeb Chrome
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari
MacOS: Desktop
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: