You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Context and feature:
Adding a clear function that erases everything stored on the JSON file. This will improve user interaction by making it easier to do multiple portfolios.
Proposed solution:
Create a new function in portfolio.go file to clear all entities in JSON file.
The text was updated successfully, but these errors were encountered:
Thank you for your interest in this project and for willing to work on adding new features! I think that the functionality you're proposing is important, however, you should be careful about how you implement it. Specifically, erasing the contents of the JSON file will likely cause an error because the program will detect it but will throw an error when attempting to read its contents. Instead, I propose that you give the user the option to delete the portfolio file completely and avoid any potential issues. You can do this in many different ways, this could be a feature specific to the web application in streamlit, where the user can click a button and the python program deletes the file. OR, you can add the functionality to Golang and make it accessible using the existing command-line interface. You'll then have to run the correct go command using python and send in the appropriate arguments. Please feel free to continue this conversation here if you need help or clarifications. Also, make sure to create a fork with all the changes you're proposing and then you can open a PR so we can further review and discuss this. Thanks again!
Context and feature:
Adding a clear function that erases everything stored on the JSON file. This will improve user interaction by making it easier to do multiple portfolios.
Proposed solution:
Create a new function in portfolio.go file to clear all entities in JSON file.
The text was updated successfully, but these errors were encountered: