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

Add image support to more brokerages #5

Open
vkruoso opened this issue Aug 9, 2015 · 1 comment
Open

Add image support to more brokerages #5

vkruoso opened this issue Aug 9, 2015 · 1 comment
Milestone

Comments

@vkruoso
Copy link
Owner

vkruoso commented Aug 9, 2015

Today only two brokerages have images available. We shall add support for at least the most popular ones. This includes finding out their number in the system.

@vkruoso vkruoso added this to the 1.0 milestone Aug 9, 2015
@vkruoso
Copy link
Owner Author

vkruoso commented Feb 12, 2017

There was actually a problem with the implementation. The number is the user ID in the brokerage and not its ID in the TD system. We are now searching using the brokerage name (implemented at 62333ed).

vkruoso added a commit that referenced this issue Feb 12, 2017
Relates to issue #5.

Signed-off-by: Vinicius Kwiecien Ruoso <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant