-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
[$250] IOU - Distance tab flickering to manual & back to distance every time when add destination #46346
Comments
Triggered auto assignment to @Christinadobrzyn ( |
We think that this bug might be related to #vip-vip |
@Christinadobrzyn FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors |
I can reproduce this - only seems to be affecting Android web app. I think this can be external. I think this might be part of Collect since it's a distance expense. |
Job added to Upwork: https://www.upwork.com/jobs/~01b7f69f2e00762143 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @mananjadhav ( |
waiting on proposals |
I'll see if someone at SWM would want this. https://expensify.slack.com/archives/C04878MDF34/p1722447232998509 |
Hi, I’m Tymoteusz from Software Mansion, the expert agency, and I’m going to be working on this task. |
Awesome! Thank you @cdOut! I've assigned you to the GH. |
@cdOut So can external contributors submit a proposal and get paid from upwork? |
@VincentCorleone I'm taking over this issue so no, there is no need for proposals. |
I'm still figuring it out, PR should be up today or tomorrow. |
Hi, @cdOut, I'm just checking on this to see if you have any specific updates - otherwise, it looks like the PR is in the works! |
No update needed, PR with the patch is being tested by Manan. |
Triggered auto assignment to @thienlnam, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
Just merged the PR, but let's keep this issue open until we get the patch removed with an upstream PR |
PR deployed to production yesterday - #47885 so adding the payment date to the subject. I'm going to be ooo 9/4 - 9/10 so I'll plan on paying this when I'm back - if that's okay. Do we need a regression? Payouts due:
|
Please check the conversation in the PR itself here, this patch sadly broke another functionality that we didn't catch during testing so we had to revert it. I'm going to be working on a fix that doesn't break the tab drag changing animation so please keep this issue open until a new PR gets merged. |
Hi, wanted to update this issue. I currently have to focus on other more important issues and I will pick this issue back up early next week. |
Thanks for the update @cdOut. Is there someone else who can pick this up? or you'll pick this once you're done with the other PRs? |
I'll pick it up after I'm done with the other PRs. |
I'm back - looks like we're working on another PR? @cdOut can you provide the new PR to follow? |
It's not raised yet, I'll let you know when I create it. |
Hi, quick update - I've picked back the issue and I'm searching for a solution that won't affect the side swiping on tabs, should raise a PR for it in the coming days. |
Short update, this is a pretty convoluted issue as fixing the flicker and allowing for the swipe causes another animation in tabs to break so I'm still researching a fix that won't disturb any other parts of the flow. |
Just a heads up that I'm going to be ooo until 9/30. I'm not going to assign another BZ teammate since we're working on the fix side of this. If you need anyone please reach out in Slack for a volunteer. |
payment summary here - #46346 (comment) |
@Christinadobrzyn this was reverted and the follow up PR is still pending. No payment yet on this one. |
I'm going to be raising a follow-up PR with another fix later today. |
Thanks for the update @cdOut |
I've found some discrepancies with animations while testing on native devices, will work on correcting it throughout today. |
Hi @cdOut do you have an update? Thanks! |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 9.0.13-1
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers): gocemate+a753@gmail.com
Issue reported by: Applause - Internal Team
Action Performed:
Expected Result:
Distance tab should remain since we work on that tab
Actual Result:
Distance tab switch to manual and back to distance every time when add destination
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6554014_1722020868045.Recording__3602.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @cdOutThe text was updated successfully, but these errors were encountered: