Closed kbecciv closed 1 year ago
Triggered auto assignment to @adelekennedy (Bug
), see https://stackoverflow.com/c/expensify/questions/14418 for more details.
Platforms
in OP are ✅)Inconsistent Enter key behaviour on Split bill page vs Invite members
Root cause is the inconsistent props set for the OptionSelector functionality in both the pages WorkspaceInvitePage
and MoneyRequestParticipantsSplitSelector
In MoneyRequestParticipantsSplitSelector
, we have a prop called shouldShowConfirmButton
which takes us to the next step after selecting maximum possible members
This is absent on workspace invite members page where we have added a manual form button using FormAlertWithSubmitButton
We need to replace the form button with the button caused by shouldShowConfirmButton
We can remove the shouldShowConfirmButton
from split selector page and add an explicit button there
Inconsistent Enter behavior between Invite members page (workspace) vs Split bill page
The invite members page does not implement the footer functionality that is already present in the OptionsSelector
component, while splitting the bill (MoneyRequestParticipantsSplitSelector
) does. The OptionsSelector
component contains the necessary functionality for correctly processing the Enter key.
Streamline the implementation of WorkspaceInvitePage
with MoneyRequestParticipantsSplitSelector
to use OptionsSelector
's built-in footer functionality, this involves using the prop shouldShowConfirmButton
.
N/A
@adelekennedy Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!
reproduced, I think the behavior should be the same!
Job added to Upwork: https://www.upwork.com/jobs/~0161c0ea65c4194594
Current assignee @adelekennedy is eligible for the External assigner, not assigning anyone new.
Triggered auto assignment to Contributor-plus team member for initial proposal review - @mananjadhav (External
)
not sure why overdue was added again - commenting to remove!
Proposal Please re-state the problem that we are trying to solve in this issue. Inconsistent Enter key behaviour on Split bill page vs Invite members
What is the root cause of that problem? Root cause is the inconsistent props set for the OptionSelector functionality in both the pages WorkspaceInvitePage and MoneyRequestParticipantsSplitSelector
In MoneyRequestParticipantsSplitSelector, we have a prop called shouldShowConfirmButton which takes us to the next step after selecting maximum possible members
This is absent on workspace invite members page where we have added a manual form button using FormAlertWithSubmitButton
What changes do you think we should make in order to solve the problem? We need to replace the form button with the button caused by shouldShowConfirmButton
Results What alternative solutions did you explore? (Optional) We can remove the shouldShowConfirmButton from split selector page and add an explicit button there
I think before we proceed to review the proposals, @adelekennedy we should define consistency? I mean of the current two disparities, which one is the expected result?
great point! I would assume we want to follow the invite flow but I'll discuss in bug-zero
huh - I was just re-testing this to post and I see that the behavior is the same now. (hitting enter selects everyone on the list)
@mananjadhav I think this has been fixed
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸
@mananjadhav, @adelekennedy Whoops! This issue is 2 days overdue. Let's get this updated quick!
Just tested again on desktop prod - I think we should close this out @Nathan-Mulugeta will be due for the reporting bonus
great! Will you apply here @Nathan-Mulugeta ?
Just applied @adelekennedy
@mananjadhav @adelekennedy this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸
Offer sent on Upwork: Issue reporter: @Nathan-Mulugeta $250.00 for the bug report
Offer accepted @adelekennedy
and 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!
Action Performed:
Expected Result:
Enter key should have a consistent action on both split bill and invite members page.
Actual Result:
Enter key does different things depending on the page
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Version Number: 1.3.37-3 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 Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/545870ba-ad65-4435-80dd-1b59630bbcfe
https://github.com/Expensify/App/assets/93399543/55996d5c-48d5-4ab1-a215-e4b1797e7419
Expensify/Expensify Issue URL: Issue reported by: @Nathan-Mulugeta Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688666077963009
View all open jobs on GitHub
Upwork Automation - Do Not Edit