Closed Wesseldr closed 1 year ago
Hi, thanks for the PR! Can you let me know why you decided to close it?
Had a bug fix and opened a new PR#36
Oh, okay. Next time you can keep it in the same PR. Just push another commit to fix the bug. That will keep all the commits relevant to this change in one place.
Ah ok.. sorry I'm not that seasoned in the PR stuff. I'll keep that in mind next time. Thanks for maintaining the package! Appreciate it!
"No matter where you go, there you are" from Buckaroo Banzai
On Tue, Sep 20, 2022 at 5:50 PM Joshua Ulrich @.***> wrote:
Oh, okay. Next time you can keep it in the same PR. Just push another commit to fix the bug. That will keep all the commits relevant to this change in one place.
— Reply to this email directly, view it on GitHub https://github.com/joshuaulrich/IBrokers/pull/34#issuecomment-1252556506, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA6TWCPFGGJSE6FMBD7K563V7HMN5ANCNFSM6AAAAAAQK4OEOM . You are receiving this because you modified the open/close state.Message ID: @.***>
Implemented the
reqMatchingSymbols
call. It returns a dataframe with the matched symbols and details. see https://interactivebrokers.github.io/tws-api/matching_symbols.html Hope you find it useful to merge, I've tried to add a manual page as well.