-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Comments
Triggered auto assignment to @bfitzexpensify ( |
@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 Screen.Recording.2024-08-20.at.17.19.50.movScreen.Recording.2024-08-20.at.17.19.12.mov |
@Expensify/design @marcaaron @JmillsExpensify Do we know what the maximum possible length of numbers that can be entered into these inputs should be? ![]() |
@marcaaron Eep! 4 days overdue now. Issues have feelings too... |
This is a good observation. I'm wondering the same thing now. 😄
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 |
@shawnborton @dannymcclain Could you take a look at it when you have some time? |
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! |
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! |
Part of the OldDot Rules Migration project
Main issue: https://github.com/Expensify/Expensify/issues/413886
Feature Description
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
The text was updated successfully, but these errors were encountered: