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

[$250] [CVP] Replace the mention in the task preview with the assigned user's avatar #48409

Closed
6 tasks done
trjExpensify opened this issue Sep 2, 2024 · 17 comments
Closed
6 tasks done
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Daily KSv2 Design External Added to denote the issue can be worked on by a contributor NewFeature Something to build that is a new item.

Comments

@trjExpensify
Copy link
Contributor

trjExpensify commented Sep 2, 2024

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: v9.0.27-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: @trjExpensify
Slack conversation: https://expensify.slack.com/archives/C07HPDRELLD/p1725028109496029 (Internal)

Action Performed:

  1. Sign-up on NewDot with a new account
  2. Choose an onboarding intent (i.e "Get paid back by my employer")
  3. Complete the onboarding steps
  4. Navigate to the Concierge DM to view the onboarding tasks created

Actual Result:

Assigned tasks include a mention of the assigned user in the task preview, but it makes it very difficult to avoid navigating to the assigned user's profile when clicking the task.

RPReplay_Final1725027273.1.MP4

Expected Result:

This is working as expected, but we want to change all assigned task previews to:

  • Not include a mention
  • Use only the avatar of the assigned user instead
  • Clicking on the task preview row (incl. the avatar) will navigate to the task, not the profile.

CleanShot 2024-08-30 at 11 13 55@2x

Workaround:

Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Yes, click the right caret.

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

See above in-line.

View all open jobs on GitHub

CC: @Expensify/design @JmillsExpensify @anmurali @danielrvidal @puneetlath

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~01c485c16de75a6534
  • Upwork Job ID: 1830665670183169524
  • Last Price Increase: 2024-09-02
  • Automatic offers:
    • dukenv0307 | Reviewer | 103801280
    • dominictb | Contributor | 103801282
Issue OwnerCurrent Issue Owner: @zanyrenney
@trjExpensify trjExpensify added Daily KSv2 NewFeature Something to build that is a new item. Design labels Sep 2, 2024
Copy link

melvin-bot bot commented Sep 2, 2024

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

Copy link

melvin-bot bot commented Sep 2, 2024

Triggered auto assignment to @dannymcclain (Design), see these Stack Overflow questions for more details.

@melvin-bot melvin-bot bot added Weekly KSv2 and removed Daily KSv2 labels Sep 2, 2024
@dannymcclain
Copy link
Contributor

Also just for added clarity: this should be a global change to all tasks (not just the onboarding tasks).

@trjExpensify
Copy link
Contributor Author

Yep, all tasks!

@bernhardoj
Copy link
Contributor

Proposal

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

Show assignee avatar instead of the email for task preview.

What is the root cause of that problem?

New redesign.

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

In TaskPreview,

  1. Get the assignee avatar
const [assigneeAvatar] = useOnyx(ONYXKEYS.PERSONAL_DETAILS_LIST, {selector: personalDetails => personalDetails?.[taskAssigneeAccountID]?.avatar});
  1. Replace the assignee mention with the avatar + task title
    <RenderHTML html={isTaskCompleted ? `<completed-task>${htmlForTaskPreview}</completed-task>` : htmlForTaskPreview} />
{taskAssigneeAccountID > 0 && <Avatar
    containerStyles={[styles.mr2, styles.taskCheckboxWrapper]}
    source={assigneeAvatar}
    size={CONST.AVATAR_SIZE.SUBSCRIPT}
    avatarID={taskAssigneeAccountID}
    type={CONST.ICON_TYPE_AVATAR}
/>}
<Text style={styles.flex1}>{taskTitle}</Text>
image

@melvin-bot melvin-bot bot added Daily KSv2 and removed Weekly KSv2 labels Sep 2, 2024
@trjExpensify trjExpensify added the External Added to denote the issue can be worked on by a contributor label Sep 2, 2024
@melvin-bot melvin-bot bot changed the title [CVP] Replace the mention in the task preview with the assigned user's avatar [$250] [CVP] Replace the mention in the task preview with the assigned user's avatar Sep 2, 2024
Copy link

melvin-bot bot commented Sep 2, 2024

Job added to Upwork: https://www.upwork.com/jobs/~01c485c16de75a6534

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Sep 2, 2024
Copy link

melvin-bot bot commented Sep 2, 2024

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

@dominictb
Copy link
Contributor

Proposal

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

Assigned tasks include a mention of the assigned user in the task preview, but it makes it very difficult to avoid navigating to the assigned user's profile when clicking the task.

What is the root cause of that problem?

When the task has assignee we'll render the mention

const htmlForTaskPreview =
taskAssigneeAccountID > 0 ? `<comment><mention-user accountid="${taskAssigneeAccountID}"></mention-user> ${taskTitle}</comment>` : `<comment>${taskTitle}</comment>`;

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

We should render the avatar when taskAssigneeAccountID>0 instead of mention and preserve the task preview style (strike through when task is done)

    const [avatar] = useOnyx(ONYXKEYS.PERSONAL_DETAILS_LIST, {selector: personalDetails => personalDetails?.[taskAssigneeAccountID]?.avatar});
    const htmlForTaskPreview = `<comment>${taskTitle}</comment>`;
...
                    {taskAssigneeAccountID > 0 && <Avatar
    containerStyles={[styles.mr2]}
    source={avatar}
    size={CONST.AVATAR_SIZE.SUBSCRIPT}
    avatarID={taskAssigneeAccountID}
    type={CONST.ICON_TYPE_AVATAR}
/>}
                    <RenderHTML html={isTaskCompleted ? `<completed-task>${htmlForTaskPreview}</completed-task>` : htmlForTaskPreview} />

since the taskTitle is just the plain text, so we can use taskTitle instead of htmlForTaskPreview

What alternative solutions did you explore? (Optional)

None

Result

Screenshot 2024-09-03 at 11 04 46

@dukenv0307
Copy link
Contributor

@dominictb's proposal LGTM. We should keep the completed task style

🎀👀🎀 C+ reviewed

Copy link

melvin-bot bot commented Sep 3, 2024

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

@melvin-bot melvin-bot bot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Sep 3, 2024
Copy link

melvin-bot bot commented Sep 3, 2024

📣 @dukenv0307 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job

Copy link

melvin-bot bot commented Sep 3, 2024

📣 @dominictb 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job
Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑‍💻
Keep in mind: Code of Conduct | Contributing 📖

@dominictb
Copy link
Contributor

dominictb commented Sep 18, 2024

@zanyrenney This hit production but not sure when, no deploy message in PR #48552.

@blimpich blimpich removed the Reviewing Has a PR in review label Sep 18, 2024
@melvin-bot melvin-bot bot added the Overdue label Sep 18, 2024
@blimpich blimpich added Awaiting Payment Auto-added when associated PR is deployed to production Daily KSv2 and removed Weekly KSv2 labels Sep 18, 2024
@melvin-bot melvin-bot bot removed the Overdue label Sep 18, 2024
@blimpich
Copy link
Contributor

blimpich commented Sep 18, 2024

@zanyrenney This hit production but not sure when, no deploy message in PR #48552.

Strange. Sorry about that @dominictb. @zanyrenney looks like this is ready for payment.

@melvin-bot melvin-bot bot added the Overdue label Sep 20, 2024
Copy link

melvin-bot bot commented Sep 23, 2024

@dannymcclain, @blimpich, @zanyrenney, @dukenv0307, @dominictb Eep! 4 days overdue now. Issues have feelings too...

@dukenv0307
Copy link
Contributor

@zanyrenney we should process payment. Thanks

@melvin-bot melvin-bot bot added Daily KSv2 and removed Daily KSv2 Overdue labels Sep 24, 2024
@zanyrenney
Copy link
Contributor

payment summary

paid $250 📣 @dukenv0307 🎉 via UW.
paid $250 📣 @dominictb 🎉 via UW.

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 Daily KSv2 Design External Added to denote the issue can be worked on by a contributor NewFeature Something to build that is a new item.
Projects
Development

No branches or pull requests

7 participants