account.get_current_positions() doesn't return option positions

[jmfernandes]

account.get_current_positions() doesn't return option positions

[jmfernandes]

get_all_option_positions(info=None) should return option positions but it seems to be a bit buggy and doesnt return everything. (Theres also one for crypto positions in crypto.py)

I should include this info in readme because its confusing why robinhood has it split up on their API but all history is in a single display on their website/app

[nick-martin-24]

agreed....would be nice to access all at once. Although in my scenario I only need to access one type.

Really cool library, I like what you've done and glad I stumbled upon this

[alexonab]

Not exactly the same issue but kind of related. When no positions are open, it would be helpful if the return value is an empty list. This makes it easier to tell if there are any positions vs the current response of [None] which is a single element list.

>>> len([]) == 0 
True
>>> len([None]) == 0 
False

Currently, I have to add an extra check to not try and process the current empty response.

        positions = r.get_open_option_positions()
        for pos in positions:
            if pos is None:
                continue
            print(f"{pos}")

[jmfernandes]

@alexonab great idea and would be easy for me to implement. All I have to do it edit helper.filter() which formats the data for all functions.

[jmfernandes]

@alexonab pull the latest code from github and install it and it should have the changes where the returned lists are [ ] instead of [None] so your if not list condition should execute properly now.