Implement 4 missing API endpoints #68
Conversation
dylanmartins
left a comment
dylanmartins
left a comment
There was a problem hiding this comment.
@ConstantBqt Hey I see that the coverage decreased a bit, could you create some tests to cover these new calls?
Thanks!
|
Hi @dylanmartins, |
|
@ConstantBqt sorry for the late answer, did you push the commit with the tests? |
My bad, I didn't push it 😅 |
Hi @dylanmartins, |
| protected function sendAuthenticatedRequest($requestMethod, $uri, $options = []) | ||
| { | ||
| $request = new \GuzzleHttp\Psr7\Request($requestMethod, $uri, $options); | ||
| $headers = array_filter($options, function ($key) { |
There was a problem hiding this comment.
I did the same kind of filter for form_params in #65
I guess there might be other key to remove here too..
- createMetaPropertyOption - modifyMetaPropertyOption - syncAssetUsage - getAccount
ConstantBqt
force-pushed
the
add_new_api_endpoints
branch
from
5ea6842 to
a0e3a63
Compare
ConstantBqt
changed the title
|
Hi, I rebased the branch and removed the duplicated endpoint. |
5 participants
Create 5 new function to use missing endpoints
Update Request handler to fix an issue with json requests (Sync asset usage)