Skip to content
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

Open
marcaaron opened this issue Apr 22, 2024 · 38 comments
Assignees
Labels
Hot Pick Ready for an engineer to pick up and run with NewFeature Something to build that is a new item. Overdue Weekly KSv2

Comments

@marcaaron
Copy link
Contributor

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

Image

We'd really like this to look more like this:

Image

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:

  • Make necessary backend changes so that the system message is created.
  • Find a contributor to deprecate the old style system message once we have deprecated them completely.
@marcaaron marcaaron added the NewFeature Something to build that is a new item. label Apr 22, 2024
Copy link

melvin-bot bot commented Apr 22, 2024

Triggered auto assignment to @Christinadobrzyn (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero process steps for feature requests for more details. Please add this Feature request to a GH project, as outlined in the SO.

Copy link

melvin-bot bot commented Apr 22, 2024

⚠️ It looks like this issue is labelled as a New Feature but not tied to any GitHub Project. Keep in mind that all new features should be tied to GitHub Projects in order to properly track external CAP software time ⚠️

Copy link

melvin-bot bot commented Apr 22, 2024

Triggered auto assignment to Design team member for new feature review - @dannymcclain (NewFeature)

@dannymcclain
Copy link
Contributor

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

Username
changed the description for this group.

Username
changed the avatar of this group.

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!)

@shawnborton
Copy link
Contributor

Hmm I don't feel too strongly - I suppose the group/room description is easily viewable in the header as well as the details view, so we don't necessarily need it. But the big thing for me is that it kind of creates an inconsistency with other places, where we do say what the value changed to. For example, categories/departments on an expense:
CleanShot 2024-04-29 at 12 56 43@2x

So I think I lean towards including it, but I also don't feel strongly.

@dannymcclain
Copy link
Contributor

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.

@dubielzyk-expensify
Copy link
Contributor

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.

@marcaaron marcaaron added the Hot Pick Ready for an engineer to pick up and run with label Apr 30, 2024
@dannymcclain
Copy link
Contributor

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!

@JmillsExpensify
Copy link

Agree with showing when a description was changed.

@Christinadobrzyn
Copy link
Contributor

Just checking on this as we're missing an external/internal label. I think this can be external @marcaaron can you confirm?

@JmillsExpensify
Copy link

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

@melvin-bot melvin-bot bot added the Overdue label May 22, 2024
@dannymcclain
Copy link
Contributor

Any update on this one?

@melvin-bot melvin-bot bot removed the Overdue label May 22, 2024
@melvin-bot melvin-bot bot added the Overdue label May 31, 2024
@dannymcclain
Copy link
Contributor

@JmillsExpensify anything new for this?

@melvin-bot melvin-bot bot removed the Overdue label Jun 4, 2024
@JmillsExpensify
Copy link

JmillsExpensify commented Jun 4, 2024

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?

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

Username
changed the description for this group.

Username
changed the avatar of this group.

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!)

@dannymcclain
Copy link
Contributor

dannymcclain commented Jun 5, 2024

We landed on showing the description.

So I think we'd have something like this:

Username
renamed this group from "Danny, Jason, Shawn" to "Grand Canyon Crew".

Username
set the description of this group to "We're going to the Grand Canyon! Let's chat about all our plans and split all our expenses here in this group. It's going to be fun on a bun!"

Username
changed the avatar for this group.

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.

@JmillsExpensify
Copy link

Gotcha. For this one...

Username
renamed this group from "Danny, Jason, Shawn" to "Grand Canyon Crew".

Would this be more consistent?

Username
renamed this group to "Grand Canyon Crew" (previously "Danny, Jason, Shawn")

@dannymcclain
Copy link
Contributor

Yeah that works! I was just going off of what the current message says:
CleanShot 2024-06-05 at 13 11 41@2x

But that doesn't really matter. I agree what you've proposed is more consistent with how we handle other updates.

@JmillsExpensify
Copy link

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.

@melvin-bot melvin-bot bot added the Overdue label Jun 14, 2024
@melvin-bot melvin-bot bot added the Overdue label Sep 19, 2024
@dannymcclain
Copy link
Contributor

Not overdue, basically still just waiting for the right time.

@melvin-bot melvin-bot bot removed the Overdue label Sep 23, 2024
@melvin-bot melvin-bot bot added the Overdue label Oct 1, 2024
@JmillsExpensify
Copy link

Still held on group chats implementation.

@melvin-bot melvin-bot bot removed the Overdue label Oct 2, 2024
@JmillsExpensify JmillsExpensify changed the title Harmonize "name" updates for Rooms and Group Chats. Deprecate non-system message update. [HOLD group chats] Harmonize "name" updates for Rooms and Group Chats. Deprecate non-system message update. Oct 2, 2024
@melvin-bot melvin-bot bot added the Overdue label Oct 10, 2024
@JmillsExpensify
Copy link

Same

@melvin-bot melvin-bot bot removed the Overdue label Oct 16, 2024
@melvin-bot melvin-bot bot added the Overdue label Oct 24, 2024
@dannymcclain
Copy link
Contributor

Still on hold Melv.

@melvin-bot melvin-bot bot removed the Overdue label Oct 24, 2024
@melvin-bot melvin-bot bot added the Overdue label Nov 4, 2024
@dannymcclain
Copy link
Contributor

hold

@melvin-bot melvin-bot bot removed the Overdue label Nov 4, 2024
@melvin-bot melvin-bot bot added the Overdue label Nov 13, 2024
@JmillsExpensify
Copy link

Haha yeah same same

@melvin-bot melvin-bot bot removed the Overdue label Nov 13, 2024
@melvin-bot melvin-bot bot added the Overdue label Nov 22, 2024
@dannymcclain
Copy link
Contributor

Still holding Melv.

@melvin-bot melvin-bot bot removed the Overdue label Nov 22, 2024
@JmillsExpensify
Copy link

Same same

@melvin-bot melvin-bot bot added the Overdue label Dec 6, 2024
@dannymcclain
Copy link
Contributor

Same³

@melvin-bot melvin-bot bot removed the Overdue label Dec 9, 2024
@melvin-bot melvin-bot bot added the Overdue label Dec 18, 2024
@JmillsExpensify
Copy link

All the same

@melvin-bot melvin-bot bot removed the Overdue label Dec 18, 2024
@melvin-bot melvin-bot bot added the Overdue label Dec 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Hot Pick Ready for an engineer to pick up and run with NewFeature Something to build that is a new item. Overdue Weekly KSv2
Projects
None yet
Development

No branches or pull requests

6 participants