Table Updates - Githubissues

ConradSelig / cliStocksTracker

A command line utility for tracking a stock market portfolio. Primarily featuring high resolution braille graphs.

Apache License 2.0

55 stars 14 forks source link

Table Updates #28

Closed patrick-hovsepian closed 3 years ago

patrick-hovsepian commented 3 years ago

created some functions for common print methods
print total cost basis of shares
print total G/L of portfolio
Add average share price to table

ConradSelig commented 3 years ago

Another quick note, you have cell_width defined in print_table() which does control how spaced out the table it - but it doesn't have any effect on the text from print_portfolio_summary, leaving the Totals: line out of sync with the table if cell_width is != 13.

If you actually wanted to convert cell_width to a class variable as table_cell_width so you don't have to pass it around to every function, I would not be opposed.