peer review 1 - Githubissues

RussDim / CIE-Casual-Investor-Essentials

A Dash app providing quick glance price data and news to the casual investor

https://cie-casual-investor-essentials.onrender.com/

MIT License

0 stars 0 forks source link

peer review 1 #1

Open YHuUBC opened 1 year ago

YHuUBC commented 1 year ago

Hi Rus, it is a pleasure reviewing your app. It is a great app that is very useful to your target persona and functions well.
The latest news section is such a great design! Users would greatly appreciate such functions.
I have two minor suggestions for your reference:
1; It is difficult for me to figure out what choices I have to enter in the 'stock ticker' section as a stock layperson. Please consider providing some information about it if you have time.
2; Maybe you can create an environment for your app and guide users to install it and use this environment to run your app locally (in the Usage and Installation section).
Great job! Pleasant to review your work.

xXJohamXx commented 1 year ago

Nice work on this app, I like how you utilized two different API's to get the data! A few suggestions:

For the input text boxes, there does not seem to be any validation, i think adding a warning or restricting the input would be a worthwhile feature to add.
I think having the ability to add more than one stock to the graph would be helpful for users looking to compare different companies.

RussDim commented 1 year ago

Thank you very much for the feedback Yaou and Jonah! Your suggestions are very valuable and I will definitely implement them in the next iteration of the app.

I will add a search box where the user can enter a company or any security name and get an exchange ticker
Multi line plots will also be included in the next iteration not only for other securities, but also fir market indicators like moving averages and other signals.

MNBhat commented 1 year ago

The app looks great, and I would like to add some suggestions

for sorting tables columns, sources, and link maybe you could explore the functionality of data table which provides the option of sorting rows rather than having a separate sorting element and updating manually.
Having an option of which new source to select can help in case there are too many news articles in a time frame
adding a multi-line plots for comparison against various stocks would be a great additionally