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 for payment 2023-08-09] Remove type-specific avatars from rooms and threads #19264

Closed
shawnborton opened this issue May 19, 2023 · 37 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering Improvement Item broken or needs improvement. Internal Requires API changes or must be handled by Expensify staff

Comments

@shawnborton
Copy link
Contributor

shawnborton commented May 19, 2023

We currently have a mix of type-specific avatars for things like the #admins, #domains, etc rooms. However, threads just use the same icon as the workspace or the parent chat they were in.
image

Let's standardize and get rid of room-specific icons. All icons should either be the workspace avatar or the DM/Group chat:
image

Note that we'll want to add a lock icon next to the room name for private/#admins rooms. This also means we'll remove the workspace badge from the LHN as well and only show it in the chat header:
image

cc @JmillsExpensify @chiragsalian @MitchExpensify @grgia @stitesExpensify can one of you take this from here?

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~0165c83124b93ca089
  • Upwork Job ID: 1664763778988531712
  • Last Price Increase: 2023-06-02
@grgia grgia self-assigned this May 22, 2023
@grgia
Copy link
Contributor

grgia commented May 22, 2023

I got this one!

@grgia
Copy link
Contributor

grgia commented May 22, 2023

@shawnborton could you explain what you mean by This also means we'll remove the workspace badge from the LHN as well and only show it in the chat header:?

image Are we just removing the pill entirely?

@grgia
Copy link
Contributor

grgia commented May 22, 2023

Also, do we want to keep the archived room one?
image

@grgia
Copy link
Contributor

grgia commented May 22, 2023

One more question- how about domain rooms?
image

@grgia grgia added the Weekly KSv2 label May 22, 2023
@shawnborton
Copy link
Contributor Author

For archived rooms, I think we just keep the name as it is with (archived)

For domain rooms, maybe we just use the default workspace avatars where we use the beginning letter of the domain?

@JmillsExpensify
Copy link

I wanted to quickly confirm that we're removing the pills entirely, right?

@shawnborton
Copy link
Contributor Author

That's my understanding.

@grgia
Copy link
Contributor

grgia commented Jun 2, 2023

Started this one

@melvin-bot melvin-bot bot removed the Overdue label Jun 2, 2023
@grgia
Copy link
Contributor

grgia commented Jun 2, 2023

Just confirming, we will be keeping the circular trash icon? @shawnborton

@chiragsalian chiragsalian added Engineering Improvement Item broken or needs improvement. Internal Requires API changes or must be handled by Expensify staff labels Jun 2, 2023
@melvin-bot
Copy link

melvin-bot bot commented Jun 2, 2023

Job added to Upwork: https://www.upwork.com/jobs/~0165c83124b93ca089

@melvin-bot
Copy link

melvin-bot bot commented Jun 2, 2023

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

@melvin-bot
Copy link

melvin-bot bot commented Jul 5, 2023

This issue has not been updated in over 15 days. @grgia, @0xmiroslav 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!

@michaelhaxhiu michaelhaxhiu changed the title [HOLD for payment 2023-08-01] Remove type-specific avatars from rooms and threads [HOLD for payment 2023-08-09] Remove type-specific avatars from rooms and threads Aug 3, 2023
@michaelhaxhiu michaelhaxhiu removed the Bug Something is broken. Auto assigns a BugZero manager. label Aug 3, 2023
@michaelhaxhiu michaelhaxhiu removed their assignment Aug 3, 2023
@michaelhaxhiu michaelhaxhiu added the Bug Something is broken. Auto assigns a BugZero manager. label Aug 3, 2023
@melvin-bot
Copy link

melvin-bot bot commented Aug 3, 2023

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

@melvin-bot melvin-bot bot added Daily KSv2 and removed Weekly KSv2 labels Aug 3, 2023
@michaelhaxhiu michaelhaxhiu self-assigned this Aug 3, 2023
@Expensify Expensify deleted a comment from melvin-bot bot Aug 3, 2023
@michaelhaxhiu
Copy link
Contributor

Note: I'm preparing to go OOO for ~2 weeks, and need a BZ buddy to oversee this while I'm gone so that payment gets sent to @0xmiroslav. Thanks in advance for your help.

As a quick recap:

  • This job is already complete
  • But @0xmiroslav needs a little time to sort out some things with their personal bank
  • So, I set an artificial deadline of 08/09 to remind us to check if it's ready to pay & close.
  • If Miki comments before that requesting payment, you can issue it immediately because the job is done!
  • No one else needs payment here.

@melvin-bot melvin-bot bot added the Overdue label Aug 7, 2023
@MitchExpensify
Copy link
Contributor

Thanks @michaelhaxhiu! Reminder set to pay out on 8/9 👍

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Aug 7, 2023
@MitchExpensify
Copy link
Contributor

Hi @0xmiroslav ! Just waiting on you to accept this offer to issue you payment https://www.upwork.com/nx/wm/offer/25771950

@melvin-bot melvin-bot bot added Overdue and removed Overdue labels Aug 9, 2023
@grgia
Copy link
Contributor

grgia commented Aug 16, 2023

Not overdue, waiting on payment finalization. Were you able to accept the offer @0xmiroslav ?

@melvin-bot melvin-bot bot removed the Overdue label Aug 16, 2023
@MitchExpensify
Copy link
Contributor

MitchExpensify commented Aug 16, 2023

I believe @0xmiroslav is sorting out a payment issue on their end Internal Slack thread for reference

@melvin-bot melvin-bot bot added the Overdue label Aug 21, 2023
@michaelhaxhiu
Copy link
Contributor

I'm back from OOO and can self assign this, just want to double check that it's not paid yet. Seems we are going to pay another individual on behalf of @0xmiroslav (and they'll sort it out from there). Is that right?

@0xmiroslav can you please confirm in newDot, where I DM'ed you about this?

@melvin-bot melvin-bot bot removed the Overdue label Aug 21, 2023
@MitchExpensify
Copy link
Contributor

Fwiw @michaelhaxhiu, I wasn't sure the best route for payment here but that seems to have been resolved based on this comment https://expensify.slack.com/archives/C01SKUP7QR0/p1692226249296529?thread_ts=1691559426.466079&cid=C01SKUP7QR0

@melvin-bot melvin-bot bot added the Overdue label Aug 24, 2023
@michaelhaxhiu
Copy link
Contributor

I invited the friend of @0xmiroslav (who he told me via DM) to issue the $1k payment. Once that's accepted, I'll pay & close.

@melvin-bot melvin-bot bot removed the Overdue label Aug 24, 2023
@michaelhaxhiu
Copy link
Contributor

paid. closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 Engineering Improvement Item broken or needs improvement. Internal Requires API changes or must be handled by Expensify staff
Projects
None yet
Development

No branches or pull requests

7 participants