Skip to content

[HOLD for payment 2024-08-09] [$250] "Leave" option icon should have "exit" icon instead of "thread" icon #46248

Closed

Description

@m-natarajan

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Version Number: 9.0.12-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @puneetlath
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1721922103779509

Action Performed:

  1. Go to workspace list
  2. Click 3 dot menu of the workspace where user is not an admin

Expected Result:

We should be using the exit icon for the "leave" option on workspaces

Actual Result:

Now it is a leave thread icon

Workaround:

unknown

Platforms:

Which of our officially supported platforms is this issue occurring on?

  • Android: Native
  • Android: mWeb Chrome
  • iOS: Native
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

CleanShot 2024-07-25 at 17 15 51@2x

image (12)

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01df4ffa2e8434c8ce
  • Upwork Job ID: 1816829296583393158
  • Last Price Increase: 2024-07-26
Issue OwnerCurrent Issue Owner: @anmurali

Activity

  1. added
    BugSomething is broken. Auto assigns a BugZero manager.
    on Jul 26, 2024
  2. melvin-bot commented on Jul 26, 2024

    @melvin-bot

    Triggered auto assignment to @anmurali (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

  3. daledah commented on Jul 26, 2024

    @daledah
    Contributor

    Proposal

    Please re-state the problem that we are trying to solve in this issue.

    Now it is a leave thread icon

    What is the root cause of that problem?

    We are using ChatBubbles in:

    icon: Expensicons.ChatBubbles,

    What changes do you think we should make in order to solve the problem?

    Should use Exit icon in this, what we used in leave room:
    image

  4. FitseTLT commented on Jul 26, 2024

    @FitseTLT
    Contributor

    Proposal

    Please re-state the problem that we are trying to solve in this issue.

    "Leave" option icon should have "exit" icon instead of "thread" icon

    What is the root cause of that problem?

    We are setting the ChatBubbles icon here

    icon: Expensicons.ChatBubbles,

    What changes do you think we should make in order to solve the problem?

    We should change it to

                        icon: Expensicons.Exit,
    
    

    What alternative solutions did you explore? (Optional)

  5. added
    ExternalAdded to denote the issue can be worked on by a contributor
    on Jul 26, 2024
  6. melvin-bot commented on Jul 26, 2024

    @melvin-bot
  7. changed the title [-]"Leave" option icon should have "exit" icon instead of "thread" icon[/-] [+][$250] "Leave" option icon should have "exit" icon instead of "thread" icon[/+] on Jul 26, 2024
  8. added
    Help WantedApply this label when an issue is open to proposals by contributors
    on Jul 26, 2024
  9. melvin-bot commented on Jul 26, 2024

    @melvin-bot

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

  10. mollfpr commented on Jul 26, 2024

    @mollfpr
    Contributor

    The proposal from @daledah looks good to me!

    🎀 👀 🎀 C+ reviewed!

  11. melvin-bot commented on Jul 26, 2024

    @melvin-bot

    Triggered auto assignment to @francoisl, see https://stackoverflow.com/c/expensify/questions/7972 for more details.

  12. 21 remaining items

  13. anmurali commented on Aug 14, 2024

    @anmurali

    @daledah - can you pls link me your Upwork profile?

  14. daledah commented on Aug 14, 2024

    @daledah
    Contributor
  15. melvin-bot commented on Aug 19, 2024

    @melvin-bot

    @francoisl, @anmurali, @mollfpr, @daledah Huh... This is 4 days overdue. Who can take care of this?

  16. melvin-bot commented on Aug 21, 2024

    @melvin-bot

    @francoisl, @anmurali, @mollfpr, @daledah 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

  17. mollfpr commented on Aug 22, 2024

    @mollfpr
    Contributor

    @anmurali Could you give the payment summary? Thank you!

  18. melvin-bot commented on Aug 23, 2024

    @melvin-bot

    @francoisl, @anmurali, @mollfpr, @daledah 8 days overdue is a lot. Should this be a Weekly issue? If so, feel free to change it!

  19. melvin-bot commented on Aug 27, 2024

    @melvin-bot

    @francoisl, @anmurali, @mollfpr, @daledah 12 days overdue. Walking. Toward. The. Light...

  20. anmurali commented on Aug 28, 2024

    @anmurali

    Payment summary

  21. daledah commented on Aug 28, 2024

    @daledah
    Contributor

    @anmurali Thanks, offer accepted

  22. anmurali commented on Aug 28, 2024

    @anmurali

    @mollfpr feel free to request via New Dot.

  23. garrettmknight commented on Aug 29, 2024

    @garrettmknight
    Contributor

    $250 approved for @mollfpr

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Awaiting PaymentAuto-added when associated PR is deployed to productionBugSomething is broken. Auto assigns a BugZero manager.DailyKSv2ExternalAdded to denote the issue can be worked on by a contributor

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions