Skip to content
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

Completed technical test #3

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Completed technical test #3

wants to merge 5 commits into from

Conversation

deepu9
Copy link

@deepu9 deepu9 commented Jan 21, 2024

Thanks for helping me out with my questions. Please find the completed technical test. Looking forward to hearing from you.

I have used Poetry, a package management tool. Added all the necessary packages into pyproject.toml file. Also, setup a script to run the server.

Packages

  1. Poetry - Package management tool
  2. FastAPI - Python framework to build API's
  3. yfinance - To get the finance data.

Folder structure
adapters - Classes to segregate the data feed logic. It get's the data feed from yahoo finance.
models - Part of ORM. A class for every table in the database. All the columns are class properties.
database.py - Has the logic to setup a database and it's tables. Also, to remove the database and tables, when the server is closed.
index.py - Entry point to the server. This has the API endpoints defined.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant