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.56k stars 2.9k forks source link

Auth GetEmailByAccountID returned an error on click of message #23663

Closed kavimuru closed 1 year ago

kavimuru commented 1 year 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!


Action Performed:

  1. click on FAB icon and select Split bill option
  2. enter amount and click Next button
  3. enter email which is not in list
  4. select that user and click on Next button
  5. click on that user from split bill list
  6. click on Message button which will redirect to report screen

    Expected Result:

    should not show any error

    Actual Result:

    Auth GetEmailByAccountID returned an error appears

    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?

Version Number: 1.3.46-0 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://github.com/Expensify/App/assets/43996225/c9586650-34ab-47d9-b565-3099db4af839

https://github.com/Expensify/App/assets/43996225/a5eb38ce-4a73-4ce5-b7e5-977857a2b6c8

Expensify/Expensify Issue URL: Issue reported by: @gadhiyamanan Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1690279806141209

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01a0ddbbb1696d85e7
  • Upwork Job ID: 1684323419830935552
  • Last Price Increase: 2023-07-26
melvin-bot[bot] commented 1 year ago

Triggered auto assignment to @MitchExpensify (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

melvin-bot[bot] commented 1 year ago

Bug0 Triage Checklist (Main S/O)

MitchExpensify commented 1 year ago

I think this will need to be internal because its probably because we're not creating accounts that don't exist the first time a bill is split. Labeling Internal

melvin-bot[bot] commented 1 year ago

Job added to Upwork: https://www.upwork.com/jobs/~01a0ddbbb1696d85e7

melvin-bot[bot] commented 1 year ago

Triggered auto assignment to Contributor Plus for review of internal employee PR - @0xmiroslav (Internal)

melvin-bot[bot] commented 1 year ago

@MitchExpensify, @0xmiroslav Huh... This is 4 days overdue. Who can take care of this?

0xmiros commented 1 year ago

@MitchExpensify I think there's something to fix in frontend as well. Can we open this to contributors or add Engineering to pull up engineer?

MitchExpensify commented 1 year ago

I think there's something to fix in frontend as well. Can we open this to contributors or add Engineering to pull up engineer?

Sounds like a plan to me! Adding the help wanted label

c3024 commented 1 year ago

PR 23802 disabled clicking on an optimistically generated user for profile page. So I think there is nothing required on front-end for this.

melvin-bot[bot] commented 1 year ago

Triggered auto assignment to @tgolen (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

MitchExpensify commented 1 year ago

PR 23802 disabled clicking on an optimistically generated user for profile page. So I think there is nothing required on front-end for this.

@tgolen does this sounds correct to you? (In which case we actually do not need any contributor for this)

tgolen commented 1 year ago

I agree. It sounds like we can simply close this out since it won't be a flow that is possible to trigger moving forward.