Skip to content

Commit

Permalink
Merge pull request #660 from facetoe/jteitelbaum/issue_658_deprecate_…
Browse files Browse the repository at this point in the history
…zopim

Fixes issue #658 deprecate zopim domain on Chat APIs
  • Loading branch information
cryptomail authored Oct 16, 2024
2 parents dfb8ab3 + 31d69d7 commit dbc48d9
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions zenpy/lib/api.py
Original file line number Diff line number Diff line change
Expand Up @@ -403,7 +403,8 @@ def _check_response(self, response):

def _build_url(self, endpoint, api_prefix=None):
""" Build complete URL """
if not issubclass(type(self), ChatApiBase) and not self.subdomain:
# Oct 29, 2024 we will require subdomain to be defined and NOT allow Zopim only requests.
if not self.subdomain:
raise ZenpyException(
"subdomain is required when accessing the Zendesk API!")

Expand Down Expand Up @@ -2116,8 +2117,7 @@ def __init__(self, config, endpoint, request_handler=None):
super(ChatApiBase, self).__init__(config,
object_type='chat',
endpoint=endpoint)
self.domain = 'www.zopim.com'
self.subdomain = ''
self.api_prefix = "api/v2/chat"
self._request_handler = request_handler or ChatApiRequest
self._object_mapping = ChatObjectMapping(self)
self._response_handlers = (DeleteResponseHandler,
Expand Down

0 comments on commit dbc48d9

Please sign in to comment.