paulinakhew / web_trader Goto Github PK

• Find a replacement name
• Replace all occurences of lq and lookupandquote with the new name

• Remove and replace all links to contact.html with "https://paulinakhew.me/"

In the functions that involve db work, a list is passed as a param:

return_list = (last_price, brokerage_fee, current_balance, trade_volume, agg_balance, username, ticker_symbol, current_number_shares)

this should be converted into its own class in a different file that is imported at the top of models. This would also help with testing as we can use an __eq__ function

https://docs.intrinio.com/documentation/api_v2/getting_started
this one also looks promising

• apply to all pages
  • Admin menu
  • Buy and sell page
  • Buy
  • Contact
  • Create
  • Dashboard
  • Leaderboard
  • Login
  • Lookup
  • Lookup and quote
  • Menu
  • Quote
  • Sell

The current API key has been revoked. I'll need to set up the new one as an env var

• this issue is similar to this one from my OutWeather Flask app
• Fix bug on all pages 🐛
  • Symbol Lookup
  • Stock Quote
  • Buy
  • Sell

• flask supports pagination!
• check out this blog post on how to do that
• @myles said this link is better
• Implement the pagination

• this issue just has to do with code readability
• in the future, when building this out, each function should be properly labeled and its purpose should be easy to identify
• this website has a lot of popular docstrings formats
• Add docstrings to all functions in model.py
• Add docstrings to all functions in controller.py
• Add line separations within functions
• remove unnecessary commented out code
• implement flake8

• Add to sqlite3 database
• Display on transaction history
• Display the time that the last price of the stock was called

• Photos
• Descriptions of each page
• talk about the database (sqlite3)
• Create instructions on how to run
• refer to this other README.md file that I created for OutWeather

• this Stack Overflow question shows you how to delete an entry within a Flask app
• Create a button in the dashboard

• if the user tries this, the user fails the unique constraint in the database
  
• find out a way to fix this so that the app catches it
• Add an alert and redirect to the create user page
• probably needs try and except statements to work

This is what the dashboard currently looks like. Users should be able to see their current account balance at the very top.

• Create an SQL Query that displays the user's account balance
• Format the balance
• I could potentially humanize the numbers using Flask-Humanize

• Use django if and else statements
  • Add to holdings table
  • Add to transactions table

• Make the redirect
• this is the same issue that I completed earlier in the OutWeather app here

• Add dollar signs to the value columns

• If the user enters a word in the "no. of shares" field for example, the app crashes
• Fix this issue throughout the application
• If the user enters an invalid input, add an alert

• I queried the database after I finished a session as paulinakhew and it still had my account under the current_user even after I signed out
• Create a function that functions similarly to seed.py that resets the current user to the original
• this issue is related to #14 since I need to make sure that the current user is always the seed user

• This method of url building is more adaptable than hard-coding urls
• Check out this url building section on the Flask documentation
• This stackoverflow question is more useful than the documentation
• Use this throughout the app

The best way to double check to find all the links would be to CMD + F on each page and search for href="/

https://www.codementor.io/@engineerapart/getting-started-with-postgresql-on-mac-osx-are8jcopb#i-introduction

The Time Purchased column displays None for sold stocks

• Fix this bug 🐛

It currently displays this error

• Add a catch

After creating an account, the app does not redirect to the main menu page. Instead it stays on the same page with an alert that you should create a different account. The newly created account exists in the system.

• Fix this bug 🐛

• Add a boolean that catches whether or not the user exists
• Add if statements to login.html and nest the alert inside

https://www.alphavantage.co/documentation/

This seems like it could be a good substitute for the markitondemand API

• the footer is overflowing into the main content
• Fix this issue

• this is how to do it

• I already signed up for a free Intrinio account
• make sure to use the Sandbox API key, which is free and has limited information
  https://docs.intrinio.com/documentation/api_v2/getting_started

• Switch to this page

this looks like a really good resource

https://devcenter.heroku.com/articles/sqlite3

SQLite3 doesn't work with Heroku

https://nitratine.net/blog/post/how-to-hash-passwords-in-python/
I'll need to hash the passwords before inserting into the table for security.

• this is a better way to code user logouts than just immediately redirecting to the login page and changing the current_user in the SQLite3 database
• Add the /logout page
• Add an alert to the /login page once the user has logged out

Right now it just raises the exception and the app errors out

It currently looks like this:

• Find a pretty Bootstrap template
• Change the HTML/CSS

• if it isn't required, it sends the input to the function and the app crashes
• Fix this bug 🐛

• depending on which brokerage, you will be charged an amount of money when buying and selling stocks
• this is how brokerage fees work
• un-hardcode the brokerage fees in the buy and sell functions

Hi Paulina!

I tried out this app, and had an issue when rendering the menu page:

I got a 403 error attempting to fetch this image. It looks like it is hosted on a third party storage platform, and doesn't have public read permissions set.

• https://blog.miguelgrinberg.com/post/the-flask-mega-tutorial-part-v-user-logins
• check out the flask-login extension
• Use flask-login
• Add flask-login to pip requirements

#1

• Make a static directory for Flask to read the images from
• Do this for the LinkedIn and GitHub logos
• New York bridge photo in contact.html

• refer to this medium article on scraping wikipedia

• In dashboard.html, the value of the holdings should be the most recent value
• Call the api of all of the stocks and return it to the page

• I can probably implement this by checking if the current user equals the seed user
• if that happens, I can return a redirect to the login page
• Fix this bug 🐛

Seems like it's down, which breaks the whole app.

• Find an alternative website to use 😢

• how to test functions with db operations
• how to mock

I should be using f strings since they're easier to read

• intro to data visualization in python

• it would be visually appealing if either the web trader menu or the dashboard had a graph of the price over time with respect to the stock they own the most quantity of

• find out how to get the data for prices in real-time

• learn how to visualize data

• implement the feature 🤩

• https://blog.ruanbekker.com/blog/2017/12/14/graphing-pretty-charts-with-python-flask-and-chartjs/

• this website shows you how to test your code

• installing and getting started with pytest

current user can be attained by calling the current_user() function so passing the username in as a parameter is deprecated