Skip to content

Workspace- Error message isn't displayed when edit a Workspace name to <b></b> #12966

Closed

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!


Issue found when executing PR #11465

Action Performed:

  1. Launch the app
  2. Log in with any account
  3. Go to Setting - Workspace
  4. Edit a Workspace name to and click Save.

Expected Result:

The error is shown as Invalid Workspace name

Actual Result:

Error message isn't displayed when edit a Workspace name to

Workaround:

Unknown

Platform:

Where is this issue occurring?

  • Web
  • iOS
  • Android
  • Desktop App
  • Mobile Web

Version Number: 1.2.30.0

Reproducible in staging?: Yes

Reproducible in production?: Yes

Email or phone of affected tester (no customers): any

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos: Any additional supporting documentation

RPReplay_Final1669227403.1.MP4

Expensify/Expensify Issue URL:

Issue reported by: Applause - Internal Team

Slack conversation:

View all open jobs on GitHub

Activity

  1. added
    BugSomething is broken. Auto assigns a BugZero manager.
    on Nov 23, 2022
  2. melvin-bot commented on Nov 23, 2022

    @melvin-bot

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

  3. Puneet-here commented on Nov 23, 2022

    @Puneet-here
    Contributor
  4. self-assigned this
    on Nov 23, 2022
  5. luacmartins commented on Nov 23, 2022

    @luacmartins
    Contributor

    Gonna pick this one up

  6. youssef-lr commented on Nov 23, 2022

    @youssef-lr
    Contributor

    This PR hasn't been merged yet which should fix this. https://github.com/Expensify/Web-Expensify/pull/35548.

  7. luacmartins commented on Nov 23, 2022

    @luacmartins
    Contributor

    Thanks for linking that issue @youssef-lr! I added this issue to the list of fixed issues on the PR you linked. I'll mark this as in review and we can retest once the Web-E PR is merged!

  8. self-assigned this
    on Nov 24, 2022
  9. ctkochan22 commented on Nov 24, 2022

    @ctkochan22

    Should we just close this in favor of #12268 ? Essentially the same bug, just worded differently I think

  10. luacmartins commented on Nov 24, 2022

    @luacmartins
    Contributor

    Sounds good to me. Closing in favor of #12268. Thanks for the input!

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

Metadata

Metadata

Labels

BugSomething is broken. Auto assigns a BugZero manager.DailyKSv2EngineeringReviewingHas a PR in review

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions