Suggesting a fix to the platformclient so a region can be specified t…

[rweeber-coveo]

This pull request is to review a simple fix to the platformclient.py and source.py classes that allow for a region parameter to be passed into the constructor. There is a default constructor value for region so that existing adoption of the SDK will continue to function. This is not a breaking change.

[rweeber-coveo]

The build failed due to something I think related to this bug: pypa/setuptools#4483. It's not something I can address on my end.

[louis-bompart]

Hi @rweeber-coveo , if you can look at why the CI is failing and fix that in another PR, that'd be much appreciated.
I won't be able to merge those changes until then.

[alexprudhomme]

I don't think https://api-us.cloud.coveo.com/push/v1/organizations/ is equal to https://api.cloud.coveo.com/push/v1/organizations/.

I can't find mentions of -us
https://docs.coveo.com/en/2976/manage-an-organization/deployment-regions-and-strategies

[evgenyfadeev]

Perhaps this method might be better:

source = Source('my_key', 'my_org', 
                             platform_push_api_domain='my_push_domain', 
                             platform_rest_api_domain='my_rest_domain')

Here is a PR: #42