[$500] Workspace - System message for workspace description edit shows from "" to "x"

[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: v1.4.42-1
Reproducible in staging?: y
Reproducible in production?: n
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
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:

Action Performed:

1. Navigate to staging.new.expensify.com
2. Create a new workspace.
3. Go to workspace profile.
4. Click Add description.
5. Add a description and save it.
6. Go to #admin room.

Expected Result:

The system message shows <updated the description of this workspace to "m">.

Actual Result:

The system message shows <updated the description of this workspace from "" to "x">.

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

Bug6380761_1708008341144.bandicam_2024-02-15_13-54-52-146.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit

• Upwork Job URL: https://www.upwork.com/jobs/~01890415a5997d6fe7
• Upwork Job ID: 1772400631077556224
• Last Price Increase: 2024-03-25

[github-actions]

👋 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.

[melvin-bot]

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

[kbecciv]

We think that this bug might be related #wave5-free-submitters
CC @dylanexpensify

[Julesssss]

We should fix this, but I'm removing the blocker label as it shouldn't block the release 👍

[melvin-bot]

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

[MitchExpensify]

Why would we expect this when the description was blank?:

The system message shows <updated the description of this workspace to "m">.

[melvin-bot]

@Julesssss, @MitchExpensify Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

[Julesssss]

Why would we expect this when the description was blank?:

Oh right. Good point. @kbecciv any other issue here we're missing, or can we close?

[MitchExpensify]

Oh right. Good point. @kbecciv any other issue here we're missing, or can we close?

Friendly bump @kbecciv

[kbecciv]

No other issues @MitchExpensify

[kbecciv]

@MitchExpensify Reopening the issue.
Previously, we had an empty workspace description, but now there is a default workspace description (due to a new feature). However, the issue remains unresolved.
Could you please advise whether it's preferable to open a new issue or to keep this one open as it is?

Bug6420607_1710946984357.such.mp4

[MitchExpensify]

Yeah lets solve that here, good catch. Can you please update the OP accordingly? Thanks

[situchan]

Unassigning as backend issue and no C+ required

[melvin-bot]

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

[hungvu193]

unassigning myself as this no need C+

[MitchExpensify]

Will try track down an Internal volunteer as a next step

[MitchExpensify]

No bites yet

@trjExpensify do we ever apply hot picks label to polish?

[trjExpensify]

Yep, you can add the Hot picks label and move it into the Hot picks section of the project board so anyone browsing it can see it in that column. Once it's picked up, remove the label and move it back into Polish. 👍

[melvin-bot]

This issue has not been updated in over 15 days. @chiragsalian, @MitchExpensify eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

[trjExpensify]

Why did you move this back, @MitchExpensify? @chiragsalian put up a PR here: https://github.com/Expensify/Web-Expensify/pull/41774

[chiragsalian]

Also that PR of mine went live last month

[chiragsalian]

Actually im just going to close out this issue since its done. Feel free to reopen if something is missing.