-
Notifications
You must be signed in to change notification settings - Fork 75
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 widgets code #108
Comments
I'll start doing some work related to this issue while working on the ex Account modal for issue #90 |
Hi, I am new to the project and would like to contribute. @mikev-cw , do you mean that in the code there are identical widgets repeated many times that you would like to modularize into a class? Can you give an example in the code of which widgets you would like to convert into a separate class? |
Welcome! Yes, there are some widgets that are shared across multiple pages but are created from scratch on each of them. The goal is to define widgets that we can reuse across the page to make the code more organized and avoid having to rewrite the same things over and over. An example, is |
@giuseppe-coco I think that that happens because you don't have any transactions, yet. Try to go to "Settings" and, in the "dev only" zone, do "clear and fill demo data". This will create a bunch of random transactions in the local database |
Yes that was the problem, thank you! |
Some widgets are now builded from scratch every time.
We need to standardize this part of the codebase
The text was updated successfully, but these errors were encountered: