updated per-action callback URI override naming~ (Azure#31335)

Author: juntuchen-msft

sdk/communication/azure-communication-callautomation/azure/communication/callautomation/_call_connection_client.py

@@ -245,7 +245,7 @@ def transfer_call_to_participant(
         sip_headers: Optional[Dict[str, str]] = None,
         voip_headers: Optional[Dict[str, str]] = None,
         operation_context: Optional[str] = None,
-        callback_url_override: Optional[str] = None,
+        callback_url: Optional[str] = None,
         transferee: Optional['CommunicationIdentifier'] = None,
         **kwargs
     ) -> TransferCallResult:
@@ -259,8 +259,10 @@ def transfer_call_to_participant(
         :paramtype voip_headers: dict[str, str]
         :keyword operation_context: Value that can be used to track this call and its associated events.
         :paramtype operation_context: str
-        :keyword callback_url_override: Url that overrides original callback URI for this request.
-        :paramtype callback_url_override: str
+        :keyword callback_url: Url that overrides original callback URI for this request.
+        :paramtype callback_url: str
+        :keyword transferee: Url that overrides original callback URI for this request.
+        :paramtype transferee: ~azure.communication.callautomation.CommunicationIdentifier
         :return: TransferCallResult
         :rtype: ~azure.communication.callautomation.TransferCallResult
         :raises ~azure.core.exceptions.HttpResponseError:
@@ -273,7 +275,7 @@ def transfer_call_to_participant(
             target_participant=serialize_identifier(target_participant),
             custom_context=user_custom_context,
             operation_context=operation_context,
-            callback_uri_override=callback_url_override
+            callback_uri=callback_url
         )
         process_repeatability_first_sent(kwargs)
         if transferee:
@@ -296,7 +298,7 @@ def add_participant(
         voip_headers: Optional[Dict[str, str]] = None,
         source_caller_id_number: Optional['PhoneNumberIdentifier'] = None,
         source_display_name: Optional[str] = None,
-        callback_url_override: Optional[str] = None,
+        callback_url: Optional[str] = None,
         **kwargs
     ) -> AddParticipantResult:
         """Add a participant to this call.
@@ -318,8 +320,8 @@ def add_participant(
         :paramtype source_caller_id_number: ~azure.communication.callautomation.PhoneNumberIdentifier or None
         :keyword source_display_name: Display name of the caller.
         :paramtype source_display_name: str or None
-        :keyword callback_url_override: Url that overrides original callback URI for this request.
-        :paramtype callback_url_override: str or None
+        :keyword callback_url: Url that overrides original callback URI for this request.
+        :paramtype callback_url: str or None
         :return: AddParticipantResult
         :rtype: ~azure.communication.callautomation.AddParticipantResult
         :raises ~azure.core.exceptions.HttpResponseError:
@@ -345,7 +347,7 @@ def add_participant(
             custom_context=user_custom_context,
             invitation_timeout=invitation_timeout,
             operation_context=operation_context,
-            callback_uri_override=callback_url_override
+            callback_uri=callback_url
         )
         process_repeatability_first_sent(kwargs)
         response = self._call_connection_client.add_participant(
@@ -361,7 +363,7 @@ def remove_participant(
         target_participant: 'CommunicationIdentifier',
         *,
         operation_context: Optional[str] = None,
-        callback_url_override: Optional[str] = None,
+        callback_url: Optional[str] = None,
         **kwargs
     ) -> RemoveParticipantResult:
         """Remove a participant from this call.
@@ -370,15 +372,16 @@ def remove_participant(
         :type target_participant: ~azure.communication.callautomation.CommunicationIdentifier
         :keyword operation_context: Value that can be used to track this call and its associated events.
         :paramtype operation_context: str
-        :keyword callback_url_override: Url that overrides original callback URI for this request.
-        :paramtype callback_url_override: str
+        :keyword callback_url: Url that overrides original callback URI for this request.
+        :paramtype callback_url: str
         :return: RemoveParticipantResult
         :rtype: ~azure.communication.callautomation.RemoveParticipantResult
         :raises ~azure.core.exceptions.HttpResponseError:
         """
         remove_participant_request = RemoveParticipantRequest(
             participant_to_remove=serialize_identifier(target_participant),
-            operation_context=operation_context, callback_uri_override=callback_url_override)
+            operation_context=operation_context,
+            callback_uri=callback_url)
 
         process_repeatability_first_sent(kwargs)
 

sdk/communication/azure-communication-callautomation/azure/communication/callautomation/_generated/_serialization.py

@@ -742,6 +742,8 @@ def query(self, name, data, data_type, **kwargs):
 
         :param data: The data to be serialized.
         :param str data_type: The type to be serialized from.
+        :keyword bool skip_quote: Whether to skip quote the serialized result.
+        Defaults to False.
         :rtype: str
         :raises: TypeError if serialization fails.
         :raises: ValueError if data is None
@@ -750,10 +752,8 @@ def query(self, name, data, data_type, **kwargs):
             # Treat the list aside, since we don't want to encode the div separator
             if data_type.startswith("["):
                 internal_data_type = data_type[1:-1]
-                data = [self.serialize_data(d, internal_data_type, **kwargs) if d is not None else "" for d in data]
-                if not kwargs.get("skip_quote", False):
-                    data = [quote(str(d), safe="") for d in data]
-                return str(self.serialize_iter(data, internal_data_type, **kwargs))
+                do_quote = not kwargs.get("skip_quote", False)
+                return str(self.serialize_iter(data, internal_data_type, do_quote=do_quote, **kwargs))
 
             # Not a list, regular serialization
             output = self.serialize_data(data, data_type, **kwargs)
@@ -892,6 +892,8 @@ def serialize_iter(self, data, iter_type, div=None, **kwargs):
          not be None or empty.
         :param str div: If set, this str will be used to combine the elements
          in the iterable into a combined string. Default is 'None'.
+        :keyword bool do_quote: Whether to quote the serialized result of each iterable element.
+        Defaults to False.
         :rtype: list, str
         """
         if isinstance(data, str):
@@ -909,6 +911,9 @@ def serialize_iter(self, data, iter_type, div=None, **kwargs):
                     raise
                 serialized.append(None)
 
+        if kwargs.get("do_quote", False):
+            serialized = ["" if s is None else quote(str(s), safe="") for s in serialized]
+
         if div:
             serialized = ["" if s is None else str(s) for s in serialized]
             serialized = div.join(serialized)