-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[HOLD group chats] Harmonize "name" updates for Rooms and Group Chats. Deprecate non-system message update. #40732
Comments
Triggered auto assignment to @Christinadobrzyn ( |
|
Triggered auto assignment to Design team member for new feature review - @dannymcclain ( |
@Expensify/design should we try to roll image and description updates into this as well? I'm thinking for those two, the system message could just be:
Or something like that. Basically I'm just thinking something slightly more generic/that doesn't include what the value was actually changed to. (I don't feel too strongly, just throwing it out there!) |
Hmm yeah that's a good point. I guess I don't feel super strongly about not including it. I guess I was just thinking that it may be kinda long—but I don't know if that should really be a concern. I would be totally down to include it like we would do with the room/group name. |
Yeah I don't actually mind including what the description was set to. Allows users to not having to navigate to see it too. But don't feel too strongly. |
Ok cool. In that case I would say if we incorporate that update into this, let's just show what the description was changed to! |
Agree with showing when a description was changed. |
Just checking on this as we're missing an external/internal label. I think this can be external @marcaaron can you confirm? |
@Christinadobrzyn system messages are backend. I hope you don't mind, but I'm going to take this one over as we try to finish up the initial scope of the design doc. |
Any update on this one? |
@JmillsExpensify anything new for this? |
Ah, sorry in what sense. I think I might have misunderstand this comment from you. As in, did we land on not showing update to the description, or showing them?
|
We landed on showing the description. So I think we'd have something like this:
Basically just following the same patterns we use for expenses where every change/update triggers a system message that creates a "history trail" of what's happened. |
Gotcha. For this one...
Would this be more consistent?
|
Cool, sounds good. So then I think we're good to get this one kicked off when an Expensify engineer is ready to pick it up. |
Not overdue, basically still just waiting for the right time. |
Still held on group chats implementation. |
Same |
Still on hold Melv. |
Haha yeah same same |
Still holding Melv. |
Same same |
Same³ |
All the same |
Coming from https://expensify.slack.com/archives/C05RECHFBEW/p1713436980732419
cc @shawnborton
Problem
Workspace rooms and Group chats both provide participants with ability to give a custom name and update it later. For the Group Chats case, when a name changes there is also no indication of who changed it.
Workspace rooms have this feature, but they use a different style of message that we are used to using (a.k.a. the system message format).
We'd really like this to look more like this:
Solution
Are there other system messages that have inconsistent style? Let's fix those too so that they all indicate who took the action and everyone can see who did it.
This is a new feature so needs an engineer to pick up as a small project:
The text was updated successfully, but these errors were encountered: