HIGH: Add a system message in the corresponding room when a room description is edited

Problem:
We are going to be allow anyone to edit the room's description, which is ripe for abuse.

Solution:
Post a system message from the editor whenever the the room description is changed.

Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01a15953af69385546
Upwork Job ID: 1740532798842871808
Last Price Increase: 2023-12-29

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

@garrettmknight

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

Bug0 Triage Checklist (Main S/O)

This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
This bug is not a duplicate report (check E/App issues and #expensify-bugs)
- If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead
This bug is reproducible using the reproduction steps in the OP. S/O
- If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
- If the reproduction steps aren't clear and you determine the correct steps, please update the OP.
This issue is filled out as thoroughly and clearly as possible
- Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.
I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

@aimane-chnaif

Triggered auto assignment to Contributor-plus team member for initial proposal review - @aimane-chnaif (External)

Oh shit, this actually can't be done externally, nm!! Call off the cavalry!!

Planning to add this next.

Draft PR is here: https://github.com/Expensify/Auth/pull/9939

Order of operation:

Merge this App PR to add a const for the new reportAction type: Add UPDATEROOMDESCRIPTION report action type #36726
Wait for this Web PR which removes an old welcome message test to be deployed to prod: https://github.com/Expensify/Web-Expensify/pull/40874
Merge and deploy this Auth PR: https://github.com/Expensify/Auth/pull/9939
Merge and deploy this Web PR: https://github.com/Expensify/Web-Expensify/pull/40917

The Auth PR is merged, but not deployed. The Web PR will get merged once the Auth PR is deployed. Will likely be done early next week given the merge freeze.

This is all done.

Navigation Menu

Repository navigation

HIGH: Add a system message in the corresponding room when a room description is edited #33736

Bug0 Triage Checklist (Main S/O)

9 remaining items

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

HIGH: Add a system message in the corresponding room when a room description is edited #33736

Description

Activity

melvin-bot commented on Dec 29, 2023

melvin-bot commented on Dec 29, 2023

melvin-bot commented on Dec 29, 2023

Bug0 Triage Checklist (Main S/O)

melvin-bot commented on Dec 29, 2023

quinthar commented on Dec 29, 2023

9 remaining items

puneetlath commented on Feb 7, 2024

puneetlath commented on Feb 14, 2024

puneetlath commented on Feb 16, 2024

puneetlath commented on Feb 21, 2024

puneetlath commented on Feb 29, 2024

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions