Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
3.51k stars 2.86k forks source link

Remove or update the “Chat with your account manager here” banner #51433

Open danielrvidal opened 5 days ago

danielrvidal commented 5 days ago

Problem: Chat with your account manager makes users think they are chatting with their account manager in Concierge.

Solution: Update the banner to look like button that clearly links to somewhere. And in the process, include the email and also have the link direct to the AM DM rather than the #admins room.

[Lightbulb] Need something specific? Chat with your account manager, [firstName] ([accountManagerEmail]). [Chat now] X

Image 1 shows the old UI. Image 2 shows the proposed updated UI. Please weigh in before I take it to GH with a contributor!

image image

dannymcclain commented 5 days ago

Figma file and easier to view updated mocks:

Account manager banner (Figma)

image

image

☝️ No difference between those mocks—just showing light and dark mode. Note that they mostly follow the banner style we use for when a chat is read only or archived and we replace the composer with a banner.

flaviadefaria commented 1 day ago

@danielrvidal Since cohort 1 is for customers with less than 10 users, then effectively, they won't have AMs. I'll keep this in the project board under Upcoming cohorts but if you think this can be prioritized elsewhere or if you think this should be prioritized differently lmk!

danielrvidal commented 1 day ago

That sounds good, I think this can be done purely by Contributors, so maybe I still open it up so they can work on it even if it's not going to be CRITICAL for this cohort.