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

[OldDot Rules Migration] Individual expense rules #47013

Closed
marcaaron opened this issue Aug 7, 2024 · 8 comments
Closed

[OldDot Rules Migration] Individual expense rules #47013

marcaaron opened this issue Aug 7, 2024 · 8 comments
Assignees
Labels
Engineering Monthly KSv2 NewFeature Something to build that is a new item. Reviewing Has a PR in review

Comments

@marcaaron
Copy link
Contributor

marcaaron commented Aug 7, 2024

Part of the OldDot Rules Migration project

Main issue: https://github.com/Expensify/Expensify/issues/413886

Feature Description

2024-08-07_13-49-38
2024-08-07_14-32-32
2024-08-07_14-32-32
2024-08-07_14-33-01
2024-08-07_14-33-07
2024-08-07_14-33-13
2024-08-07_14-33-21
2024-08-07_14-33-30

High Level Section: https://docs.google.com/document/d/1oLr14YhL6Y0N5g4tbozdIIrFbybBlsRA0H9I8Wm--w8/edit#bookmark=id.ycyp9y4tlna4

Detailed Section: https://docs.google.com/document/d/1oLr14YhL6Y0N5g4tbozdIIrFbybBlsRA0H9I8Wm--w8/edit#bookmark=id.ore5lrt7u7r

Manual Test Steps

TBD

Automated Tests

TBD

@melvin-bot melvin-bot bot added the Monthly KSv2 label Aug 12, 2024
@marcaaron marcaaron added Daily KSv2 Monthly KSv2 NewFeature Something to build that is a new item. and removed Monthly KSv2 Daily KSv2 labels Aug 14, 2024
Copy link

melvin-bot bot commented Aug 14, 2024

Triggered auto assignment to @bfitzexpensify (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.

@WojtekBoman
Copy link
Contributor

@Expensify/design

I have one question about the Billable default page. Currently we don't have a pattern in the app when we select an element from the list and we have to additionally confirm our selection by clicking a button. It causes some issues with setting the background color of the selected item. Usually we don't need to change the background color of the selected item because we close the page once we select an option. I'm wondering if we need this submit button on this page. If so, I'll try to find a solution, but if not, I'll implement it the same as other pages with SelectionList. I'm attaching a video with the current implementation and examples of other pages using SelectionList.

Screen.Recording.2024-08-20.at.17.19.50.mov
Screen.Recording.2024-08-20.at.17.19.12.mov

@WojtekBoman
Copy link
Contributor

@Expensify/design @marcaaron @JmillsExpensify

Do we know what the maximum possible length of numbers that can be entered into these inputs should be?

Screenshot 2024-08-20 at 17 26 36

Copy link

melvin-bot bot commented Aug 20, 2024

@marcaaron Eep! 4 days overdue now. Issues have feelings too...

@marcaaron
Copy link
Contributor Author

I'm wondering if we need this submit button on this page.

This is a good observation. I'm wondering the same thing now. 😄

Do we know what the maximum possible length of numbers that can be entered into these inputs should be?

On the API side there is no maximum. Most people will put reasonably low values because larger values are impractical i.e. it would be the same as putting any large number after a point. If we want to impose something we could use the DISABLED_MAX_EXPENSE_VALUE, but not sure we need to do anything here tbh.

@melvin-bot melvin-bot bot removed the Overdue label Aug 20, 2024
@WojtekBoman
Copy link
Contributor

@Expensify/design

I have one question about the Billable default page. Currently we don't have a pattern in the app when we select an element from the list and we have to additionally confirm our selection by clicking a button. It causes some issues with setting the background color of the selected item. Usually we don't need to change the background color of the selected item because we close the page once we select an option. I'm wondering if we need this submit button on this page. If so, I'll try to find a solution, but if not, I'll implement it the same as other pages with SelectionList. I'm attaching a video with the current implementation and examples of other pages using SelectionList.

Screen.Recording.2024-08-20.at.17.19.50.mov
Screen.Recording.2024-08-20.at.17.19.12.mov

@shawnborton @dannymcclain Could you take a look at it when you have some time?

@shawnborton
Copy link
Contributor

In the page you are referencing for billable default, you are right in that we probably don't need a Save button - we can just allow the selection of the list item to actually select something and change the form.

That being said, we keep running into small bugs where a selected item does not have a correct background color. So whether we save it immediately or not, we really need to make sure all selected list items use the appropriate background color. Let me know if that helps clear things up!

@melvin-bot melvin-bot bot added Reviewing Has a PR in review Weekly KSv2 and removed Daily KSv2 labels Aug 23, 2024
@melvin-bot melvin-bot bot removed the Weekly KSv2 label Sep 16, 2024
Copy link

melvin-bot bot commented Sep 16, 2024

This issue has not been updated in over 15 days. @marcaaron eroding to Monthly issue.

P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Engineering Monthly KSv2 NewFeature Something to build that is a new item. Reviewing Has a PR in review
Projects
Status: Done
Development

No branches or pull requests

4 participants