App: Unable to delete a contact method including capital letters

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Action Performed:

  1. Launch the app -> Login in account
  2. Navigate to Avatar > Profile > contact method
  3. Add a new contact email including capital letters i.e DERP@DERP.COM
  4. Delete the contact method you just added

Expected Result:

You should be able to delete the contact method

Actual Result:

Attempting to delete the contact method results in this error: Auth DeleteLogin returned an error

Workaround:

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

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android / native
  • Android / Chrome
  • iOS / native
  • iOS / Safari
  • MacOS / Chrome / Safari
  • MacOS / Desktop

Version Number: v1.3.14-14 Reproducible in staging?: Y Reproducible in production?: Y 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 Notes/Photos/Videos: Any additional supporting documentation

https://user-images.githubusercontent.com/47522946/236903532-b51120b8-5740-4a64-8499-ff3cde0461b8.mp4

Expensify/Expensify Issue URL: Issue reported by: @dhairyasenjaliya Slack conversation: https://github.com/Expensify/App/issues/18301#issuecomment-1538852132

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01c866a623249c0072
  • Upwork Job ID: 1658592013696098304
  • Last Price Increase: 2023-05-16

About this issue

  • Original URL
  • State: closed
  • Created a year ago
  • Comments: 36 (27 by maintainers)

Most upvoted comments

Oh shit that’s totally my bad 🙃

Ah, sorry man. @Beamanator throwing me off! 😆

FYI there was no payment needed, it was resolved internally

Sent you a contract on this job.

It could be a good idea to prevent users from entering capital letters in emails in the front end, but this will be the last step so I’m planning to worry about this later - but I’ll come back to your thoughts for sure @dhairyasenjaliya - thanks for your ideas!

FYI I started an internal discussion here to think about a migration we might have to run, making existing contact methods lowercase in the database & Onyx

Gooooooood question @dhairyasenjaliya , I will do some investigation soon, most likely it’s just a backend change though