-
-
Notifications
You must be signed in to change notification settings - Fork 64
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
Refactor sharedProps
prop in src/forms/CurrentExpenses.js
#986
Comments
Could you add an example of how it would look where it's separated? |
For example, the
So instead of having the
it could be
The code that uses
The advantage of this approach is that it's clearer what |
That's strange that |
#965 uses a prop called
sharedProps
for several components; this should be separated out into the specific props each component needs.The text was updated successfully, but these errors were encountered: