Closed Julesssss closed 1 year ago
May be related https://github.com/Expensify/App/issues/15644
Show up to 4 participant avatars in IOUPreview for split bill
The root cause is that we're not using the prop shouldStackHorizontally
here.
We need to add shouldStackHorizontally
and set size
prop to small
. Additionally, the secondAvatarStyle
prop needs to be removed. These changes need to be done here only if the isBillSplit
is true
.
None
@Julesssss Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!
Triggered auto assignment to @flaviadefaria (Bug
), see https://stackoverflow.com/c/expensify/questions/14418 for more details.
Platforms
in OP are ✅)Job added to Upwork: https://www.upwork.com/jobs/~01a91334bf09c52786
Current assignee @flaviadefaria is eligible for the External assigner, not assigning anyone new.
Triggered auto assignment to Contributor-plus team member for initial proposal review - @aimane-chnaif (External
)
Current assignee @Julesssss is eligible for the External assigner, not assigning anyone new.
The proposal from @allroundexperts looks good to me. @aimane-chnaif would you mind reviewing this before I assign it? Thanks
Proposal looks good to me too. @allroundexperts let's make sure order doesn't change when become online from offline And need to customize styles. It looks ugly without them:
https://github.com/Expensify/App/assets/96077027/4aa2960b-dc35-4428-ac0a-b7e22524486c
Proposal looks good to me too. @allroundexperts let's make sure order doesn't change when become online from offline And need to customize styles. It looks ugly without them:
Screen.Recording.2023-05-16.at.4.26.13.PM.mov
@aimane-chnaif Sure thing.
Great, let's get this done
📣 @allroundexperts You have been assigned to this job by @Julesssss! Please apply to this job in Upwork 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 📖
PR created https://github.com/Expensify/App/pull/19074
I'm OoO for 8 days so re-adding the BUG label, but keeping myself assigned. I'll be back at work on May 30th.
Triggered auto assignment to @zanyrenney (Bug
), see https://stackoverflow.com/c/expensify/questions/14418 for more details.
Platforms
in OP are ✅)Reviewing
label has been removed, please complete the "BugZero Checklist".
The solution for this issue has been :rocket: deployed to production :rocket: in version 1.3.17-5 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:
If no regressions arise, payment will be issued on 2023-06-01. :confetti_ball:
After the hold period is over and BZ checklist items are completed, please complete any of the applicable payments for this issue, and check them off once done.
As a reminder, here are the bonuses/penalties that should be applied for any External issue:
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
@aimane-chnaif can you please fill out the checklist above so that I can issue payment? Thanks!
This is not a bug bug feature request. No regression tests needed. This is part of rolling out a new feature [manual request] and comprehensive regression test will be updated for money request pages.
@allroundexperts and @aimane-chnaif I'm sending you offers in UW. $1000 + $500 bonus = $1500 total payment for each of you.
Everyone has been paid so closing this.
Problem
We introduced the IOUPreview for bill splits https://github.com/Expensify/Expensify/issues/270707. But we are only showing one avatar in the IOUPreview when a bill split is created. Instead, we should show up to 4, or 3 if more than 4 participants are included in the split.
Current
Desired
In addition, we should make sure the same order is used when the split is created while offline. (So when returning online, the order shouldn't change)
Solution
Yes please.
Upwork Automation - Do Not Edit