Adding support for 'getWorkbookPdf' endpoint of REST API#376
Adding support for 'getWorkbookPdf' endpoint of REST API#376shinchris merged 1 commit intodevelopmenttableau/server-client-python:developmentfrom
Conversation
| def id(self): | ||
| return self._id | ||
|
|
||
| @property |
There was a problem hiding this comment.
Do you remember why we did this pattern of lazy population?
There was a problem hiding this comment.
Many moons ago, it was because we didn't want to "make any calls the user didn't ask for".
Originally it populated a static list for things like Views in Workbooks; but when we needed paging for nested lists (Users in Groups), I converted everything to an on-demand function call; sometimes this is nice because you can filter or change the paging parameters of nested items. I didn't take it the whole way to making the property take request options and handling all of them to just hidden on-demand calls. Lee had some qualms about mixing requests and models, if I recall. Here, where I bind a secret function to the property was as far as we got :P
No description provided.