search

Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
3.33k stars 2.76k forks source link

[Android] mWeb/Chrome - BNP - Inserting decimal point, inconsistent behavior in mweb&Android #47074

Open IuliiaHerets opened 1 month ago

IuliiaHerets commented 1 month ago

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.18 Reproducible in staging?: Y Reproducible in production?: Y If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4836262 Issue reported by: Applause Internal Team

Action Performed:

  1. Launch app in both mweb and Android
  2. Tap fab -- submit expense
  3. Enter 23 and keep cursor between 2&3 and type dot to insert decimal point
  4. Note the behavior in mweb & Android

Expected Result:

Inserting decimal point, behavior must be consistent in mweb& Android.

Actual Result:

In Android, decimal point is inserted correctly in the same place but in mweb, decimal point is inserted after a number. Inserting decimal point, inconsistent behavior in mweb& Android.

Workaround:

Can the user still use Expensify without this being fixed? Have you informed them of the workaround?

Platforms:

Screenshots/Videos

https://github.com/user-attachments/assets/343a82d9-0c96-43f1-ac9a-fa8f2c0c5c83

View all open jobs on GitHub

melvin-bot[bot] commented 1 month ago

Triggered auto assignment to @mallenexpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

IuliiaHerets commented 1 month ago

We think that this bug might be related to #vip-vsb

IuliiaHerets commented 1 month ago

@mallenexpensify 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

MelvinBot commented 1 month ago

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

mallenexpensify commented 1 month ago

Gonna try to pawn this off on Sheena when she gets assigned another iOS bug :)

mallenexpensify commented 3 weeks ago

Oh snap.... we got an Android phone on the way to the bank! I'll be able to test soon.

melvin-bot[bot] commented 3 weeks ago

@mallenexpensify 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!

mallenexpensify commented 2 weeks ago

Need to get Android phone setup, will do next week, bumped to Daily

mallenexpensify commented 1 week ago

Unsure when in the office this week but planning to get phone setup

melvin-bot[bot] commented 1 week ago

@mallenexpensify Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

mallenexpensify commented 1 week ago

Still need to get phone setup 😿

melvin-bot[bot] commented 3 days ago

@mallenexpensify Whoops! This issue is 2 days overdue. Let's get this updated quick!