Skip to content

[HOLD for payment 2024-06-28] [HOLD for payment 2024-06-28] mWeb - Room description - When saving specific room description different error shown in stg&pro #43925

Description

@lanitochka17

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

Action Performed:

  1. Go to https://staging.new.expensify.com/
  2. Tap on a room chat
  3. Tap header --- room description
  4. Paste the @×÷==//^^^&(/>[//<[[&^+&*6>[[_&^()/₹₹*^8÷_€\€|€●■{€|♡ 65€●6
  5. Tap save

Expected Result:

Same error must be shown in staging and production

Actual Result:

When saving specific room description different error shown in staging and production

Workaround:

Unknown

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

Bug6517312_1718720478838!Screenshot_20240618-193749_Chrome

View all open jobs on GitHub

Activity

  1. added
    DeployBlockerCashThis issue or pull request should block deployment
    DeployBlockerIndicates it should block deploying the API
    on Jun 18, 2024
  2. melvin-bot commented on Jun 18, 2024

    @melvin-bot

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

  3. github-actions commented on Jun 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.
  4. lanitochka17 commented on Jun 18, 2024

    @lanitochka17
    Author

    Production:

    193800_Chrome

  5. lanitochka17 commented on Jun 18, 2024

    @lanitochka17
    Author

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

  6. bondydaa commented on Jun 18, 2024

    @bondydaa
    Contributor

    not a web blocker, looks like something changed in the error string being used.

  7. bondydaa commented on Jun 18, 2024

    @bondydaa
    Contributor

    that text comes from here https://github.com/Expensify/App/blob/main/src/components/Form/FormProvider.tsx#L146 but hasn't changed for ~6 months so something else seems to have changed.

  8. bondydaa commented on Jun 18, 2024

    @bondydaa
    Contributor

    trying to revert this commit locally to confirm 0c4cabc

  9. fedirjh commented on Jun 18, 2024

    @fedirjh
    Contributor

    @bondydaa It seems this was missed during this refactor: #42970

    validateErrors[inputID] = 'common.error.invalidCharacter';

    it should be updated as follows :

    validateErrors[inputID] = translate('common.error.invalidCharacter');
  10. 13 remaining items

  11. bondydaa commented on Jun 18, 2024

    @bondydaa
    Contributor

    tested on staging and looking good 🎉
    image

    removing blocker

  12. bondydaa commented on Jun 18, 2024

    @bondydaa
    Contributor

    I think we can close this out since it was just a regression from #42970, let me know if that's wrong @nkdengineer or @fedirjh

  13. added
    Awaiting PaymentAuto-added when associated PR is deployed to production
    and removed on Jun 21, 2024
  14. changed the title [-]mWeb - Room description - When saving specific room description different error shown in stg&pro[/-] [+][HOLD for payment 2024-06-28] mWeb - Room description - When saving specific room description different error shown in stg&pro[/+] on Jun 21, 2024
  15. melvin-bot commented on Jun 21, 2024

    @melvin-bot

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

  16. melvin-bot commented on Jun 21, 2024

    @melvin-bot

    The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.85-7 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-06-28. 🎊

    For reference, here are some details about the assignees on this issue:

    • @fedirjh requires payment (Needs manual offer from BZ)
    • @nkdengineer requires payment (Needs manual offer from BZ)
  17. changed the title [-][HOLD for payment 2024-06-28] mWeb - Room description - When saving specific room description different error shown in stg&pro[/-] [+][HOLD for payment 2024-06-28] [HOLD for payment 2024-06-28] mWeb - Room description - When saving specific room description different error shown in stg&pro[/+] on Jun 21, 2024
  18. melvin-bot commented on Jun 21, 2024

    @melvin-bot

    The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.0-9 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-06-28. 🎊

    For reference, here are some details about the assignees on this issue:

    • @fedirjh requires payment (Needs manual offer from BZ)
    • @nkdengineer requires payment (Needs manual offer from BZ)
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

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions