search

MicroStrategy / mstrio-py

Python integration for MicroStrategy
Apache License 2.0
90 stars 60 forks source link

changed cube_info call to allow list of ids #24

Closed r3dunlop closed 4 years ago

r3dunlop commented 4 years ago

Moved id query into the body of the get request for cube_info. This allows for more cubes to be in the response from a single request. The other option is to use {}".format(cube_id) in the url.

drzamich commented 4 years ago

Hello @r3dunlop, thank you for your contribution. Unfortunately, the solution proposed by you cannot work, as the /cubes/{cubeId} endpoint of the MicroStrategy REST API on which the mstrio-py is based does not allow multiple cube ids to be passed in one request. Please refer to the REST API docs to get more information about available endpoints. I will pass your suggestion as an enhancement request to the team working on the REST API.

r3dunlop commented 4 years ago

Hi @drzamich thank you for the response. I was changing the function to use a different endpoint. It looks like I PR'd a version where I modified the wrong API function. I was intending to change the cube_info function. Would that not work?