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

get_orders_by_path formats HTTP request with datetimes while TDA expects dates #177

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

get_orders_by_path formats HTTP request with datetimes while TDA expects dates #177

wants to merge 2 commits into from

Commits on Feb 20, 2021

  1. get_orders_by_path uses datetimes while expecting dates 

    While debugging
https://tda-api.readthedocs.io/en/stable/client.html#accessing-existing-orders
I noticed that when you specify no arguments it would not return any
data.  After digging in I saw that
https://developer.tdameritrade.com/account-access/apis/get/orders-0 only
accepts yyyy-MM-dd format
    @silasb
    silasb committed Feb 20, 2021
    Configuration menu
    Copy the full SHA
    79b090f View commit details
    Browse the repository at this point in the history

  2. Getting tests to pass

    @silasb
    silasb committed Feb 20, 2021
    Configuration menu
    Copy the full SHA
    9f1a901 View commit details
    Browse the repository at this point in the history