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.

Sign up for GitHub

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

Jump to bottom

Execute trades job issue 145 #174

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

Execute trades job issue 145 #174

wants to merge 9 commits into from

Commits on Jun 2, 2024

  1. Add pending scope to order model and add test

    @americac @jonny5
    americac authored and jonny5 committed Jun 2, 2024
    Configuration menu
    Copy the full SHA
    2919e89 View commit details
    Browse the repository at this point in the history

  2. Add purchase_stock service to handle stock purchases

    @americac @jonny5
    americac authored and jonny5 committed Jun 2, 2024
    Configuration menu
    Copy the full SHA
    8d40e67 View commit details
    Browse the repository at this point in the history

  3. [MIGRATION] Change portfolio_stock shares and purchase_price to decimal

    @americac @jonny5
    americac authored and jonny5 committed Jun 2, 2024
    Configuration menu
    Copy the full SHA
    52c2547 View commit details
    Browse the repository at this point in the history

  4. Add reference to portfolio_stock and portfolio_transaction to orders

    @americac @jonny5
    americac authored and jonny5 committed Jun 2, 2024
    Configuration menu
    Copy the full SHA
    ddeb095 View commit details
    Browse the repository at this point in the history

  5. Add a class execute method to service

    @jonny5
    jonny5 committed Jun 2, 2024
    Configuration menu
    Copy the full SHA
    d23e58b View commit details
    Browse the repository at this point in the history

  6. Add stock purchase job 

    - After this job runs we will want to enqueue the `StockPricesUpdateJob`, not sure about that implementation yet
    @jonny5
    jonny5 committed Jun 2, 2024
    Configuration menu
    Copy the full SHA
    bbe21f3 View commit details
    Browse the repository at this point in the history

  7. Remove unique index on portfolio_stocks on portfolio and stock 

    - Multiple portfolio_stocks can exist for a portfolio with different purchase prices
    @jonny5
    jonny5 committed Jun 2, 2024
    Configuration menu
    Copy the full SHA
    2d82828 View commit details
    Browse the repository at this point in the history

  8. use fixture for stock

    @jonny5
    jonny5 committed Jun 2, 2024
    Configuration menu
    Copy the full SHA
    b73b345 View commit details
    Browse the repository at this point in the history

  9. Add failure state test 

    - Use mocha for stubbing
    @jonny5
    jonny5 committed Jun 2, 2024
    Configuration menu
    Copy the full SHA
    7ddd8ae View commit details
    Browse the repository at this point in the history