Skip to content

[HOLD for payment 2024-04-05] [HOLD for payment 2024-03-29] [HOLD for payment 2024-03-26] [Simplified Collect][Taxes] iOS - Taxes - App crashes tapping tax value #38502

Description

@kbecciv

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: 1.4.54-0
Reproducible in staging?: y
Reproducible in production?: n
Issue reported by: Applause - Internal Team

Action Performed:

Precondition:

  • User is an admin of Collect workspace.
  1. Open app
  2. Go to Profile > Workspaces > Collect workspace.
  3. Go to Taxes.
  4. Click Add rate.
  5. Click Value.

Expected Result:

Able to add tax value

Actual Result:

App crashes

Workaround:

n/a

Platforms:

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

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

Bug6418175_1710785068726.RPReplay_Final1710784552.1.mp4

View all open jobs on GitHub

Activity

  1. github-actions commented on Mar 18, 2024

    @github-actions
    Contributor

    👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

    1. Identify the pull request that introduced this issue and revert it.
    2. Find someone who can quickly fix the issue.
    3. Fix the issue yourself.
  2. melvin-bot commented on Mar 18, 2024

    @melvin-bot

    Triggered auto assignment to @francoisl (Engineering), see https://stackoverflowteams.com/c/expensify/questions/9980/ for more details.

  3. kbecciv commented on Mar 18, 2024

    @kbecciv
    Author

    We think that this bug might be related to #wave-control

  4. kbecciv commented on Mar 18, 2024

    @kbecciv
    Author

    @francoisl The app crashes when adding the distance rate, do we need to create a separate GH or the same root cause?

    RPReplay_Final1710786838.MP4
  5. allgandalf commented on Mar 18, 2024

    @allgandalf
    Contributor

    #38504 (comment)

    @abzokhattab , looks like the same RCA is applicable here too as you mentioned over that issue

  6. francoisl commented on Mar 18, 2024

    @francoisl
    Contributor

    I can repro on iOS, this seems to come from #37967.

    The app crashes when adding the distance rate, do we need to create a separate GH or the same root cause

    Not sure yet but it might be a different cause, the PR I linked to is specific to tax rates. Maybe it's safer to open a separate issue for now.

  7. luacmartins commented on Mar 18, 2024

    @luacmartins
    Contributor

    This was introduced #37967. This is NAB since this feature is only available on Collect policies in NewDot and users don't have access to that yet. I think we can address that as a follow up to the other tax PRs. cc @kosmydel @filip-solecki

  8. added and removed
    DeployBlockerCashThis issue or pull request should block deployment
    on Mar 18, 2024
  9. 15 remaining items

  10. added
    Awaiting PaymentAuto-added when associated PR is deployed to production
    and removed on Mar 19, 2024
  11. changed the title [-][Simplified Collect][Taxes] iOS - Taxes - App crashes tapping tax value[/-] [+][HOLD for payment 2024-03-26] [Simplified Collect][Taxes] iOS - Taxes - App crashes tapping tax value[/+] on Mar 19, 2024
  12. melvin-bot commented on Mar 19, 2024

    @melvin-bot

    Reviewing label has been removed, please complete the "BugZero Checklist".

  13. melvin-bot commented on Mar 19, 2024

    @melvin-bot

    The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.54-4 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

    If no regressions arise, payment will be issued on 2024-03-26. 🎊

  14. changed the title [-][HOLD for payment 2024-03-26] [Simplified Collect][Taxes] iOS - Taxes - App crashes tapping tax value[/-] [+][HOLD for payment 2024-03-29] [HOLD for payment 2024-03-26] [Simplified Collect][Taxes] iOS - Taxes - App crashes tapping tax value[/+] on Mar 22, 2024
  15. melvin-bot commented on Mar 22, 2024

    @melvin-bot

    The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.55-3 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

    If no regressions arise, payment will be issued on 2024-03-29. 🎊

  16. changed the title [-][HOLD for payment 2024-03-29] [HOLD for payment 2024-03-26] [Simplified Collect][Taxes] iOS - Taxes - App crashes tapping tax value[/-] [+][HOLD for payment 2024-04-05] [HOLD for payment 2024-03-29] [HOLD for payment 2024-03-26] [Simplified Collect][Taxes] iOS - Taxes - App crashes tapping tax value[/+] on Mar 29, 2024
  17. melvin-bot commented on Mar 29, 2024

    @melvin-bot

    The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.57-5 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:

    If no regressions arise, payment will be issued on 2024-04-05. 🎊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Type

No type

Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions