-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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 2024-04-22] [$500] Group - All 8 participants are shown in header on creating group #38052
Comments
Triggered auto assignment to @anmurali ( |
We think that this bug might be related to #vip-vsp |
@anmurali FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors |
ProposalPlease re-state the problem that we are trying to solve in this issue.All selected participants are displayed on the header (names) What is the root cause of that problem?The headerView component display all the available 8 participants. What changes do you think we should make in order to solve the problem?We can limit the number of participants to 5 here App/src/pages/home/HeaderView.tsx Line 78 in 2979c4f
|
ProposalPlease re-state the problem that we are trying to solve in this issue.Group - All 8 participants are shown in header on creating group What is the root cause of that problem?The root cause of the problem is here we are displaying all the participants personal details App/src/pages/home/HeaderView.tsx Line 79 in 5469198
and then in here : App/src/pages/home/HeaderView.tsx Line 81 in 5469198
we are displaying it What changes do you think we should make in order to solve the problem?We can limit it to just display 5 participants by doing this :
What alternative solutions did you explore? (Optional)N/A Result :Screen.Recording.2024-03-11.at.10.59.49.PM.mp4 |
@deetergp Before you mentioned the issue, I already had that solution. |
Still sorting this one out on the original GH. |
@deetergp I posted a comment here #37907 (comment) to explain how I think that the original GH is not solving the real issue |
Job added to Upwork: https://www.upwork.com/jobs/~019807d7b57314faa7 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @Pujan92 ( |
Proposal Issue: This issue is being caused due to text overflowing in a given area. Proposed Solution: We can extract the first five Usernames within the chat participants and only display their names. We may want to add a way to trunc the concatenated names if the text overflows. |
📣 @DinCorp777! 📣
|
I think we want to keep only 5 members in the report name |
Contributor details |
✅ Contributor details stored successfully. Thank you for contributing to Expensify! |
Current assignee @deetergp is eligible for the choreEngineerContributorManagement assigner, not assigning anyone new. |
📣 @Pujan92 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app! |
📣 @dragnoir You have been assigned to this job! |
PR ready #38568 |
PR has been merged and is currently being tested on staging. |
@Pujan92 what's the next step here? I don't see the checklist bot! |
@anmurali can you pls proceed the payment? Thank you |
@anmurali Friendly bump |
@anmurali accepted. Thank you |
Paid |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Issue found when executing PR #37907
Version Number: 1.4.50-2
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:
Expected Result:
When creating group with 8 participants, the group conversation header must display only 5 participants
Actual Result:
When creating group with 8 participants, the group conversation header displays all 8 participants
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: