-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[DO NOT MERGE] GA4 changes #31902
Draft
v-vdharmaraj
wants to merge
80
commits into
main
Choose a base branch
from
callautomation/release/ga4
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
[DO NOT MERGE] GA4 changes #31902
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
### Issues associated with this PR [User Story 3576848](https://skype.visualstudio.com/SPOOL/_workitems/edit/3576848): [Beta4][CA SDK][JS] Update SDK with the interrupt call media flag for Play api ### Describe the problem that is addressed by this PR 1.Added beta4 interruptCallMediaOperation to playToAll. 2.Unit test. ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary)
### Issues associated with this PR [User Story 3586532](https://skype.visualstudio.com/SPOOL/_workitems/edit/3586532): [JS][SDK][Beta4] Node Audio streaming Sdk ### Describe the problem that is addressed by this PR 1.StartMediaStreaming. 2.StopMediaSteaming. 3.Unit Test. 4.Media streaming events. ### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen? ### Are there test cases added in this PR? _(If not, why?)_ ### Provide a list of related PRs _(if any)_ ### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_ ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary) Co-authored-by: Vinothini Dharmaraj <[email protected]>
### Describe the problem that is addressed by this PR Added beta4 PlayStarted event in event parser. Co-authored-by: Vinothini Dharmaraj <[email protected]>
…#29719) ### Issues associated with this PR [User Story 3667737](https://skype.visualstudio.com/SPOOL/_workitems/edit/3667737): [Beta4][JS][SDK] Update the SDK according to Azure Board's comments. ### Describe the problem that is addressed by this PR HoldOptions update operationCallbackUri to operationCallbackUrl ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary)
### Issues associated with this PR [User Story 3672944](https://skype.visualstudio.com/SPOOL/_workitems/edit/3672944): [JS][SDK][BETA4] Add Prompts(List) in Recognize feature ### Describe the problem that is addressed by this PR 1.Recognize PromptList. 2.Unit Test. ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary) Co-authored-by: Vinothini Dharmaraj <[email protected]>
### Describe the problem that is addressed by this PR 1.Added beta4 audio streaming parser. 2.Unit test. ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary)
### Describe the problem that is addressed by this PR 1.Fixed streamDataParser transcription data mapping. 2.Updated unit test. ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary)
### Packages impacted by this PR ### Issues associated with this PR ### Describe the problem that is addressed by this PR ### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen? ### Are there test cases added in this PR? _(If not, why?)_ ### Provide a list of related PRs _(if any)_ ### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_ ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary) --------- Co-authored-by: Azure SDK Bot <[email protected]> Co-authored-by: Wes Haggard <[email protected]> Co-authored-by: Jeremy Meng <[email protected]> Co-authored-by: Jackson Weber <[email protected]> Co-authored-by: EmmaZhu-MSFT <[email protected]> Co-authored-by: James Suplizio <[email protected]> Co-authored-by: luc <[email protected]> Co-authored-by: Maor Leger <[email protected]> Co-authored-by: Jeremy Meng <[email protected]> Co-authored-by: KarishmaGhiya <[email protected]> Co-authored-by: Matthew Podwysocki <[email protected]> Co-authored-by: Minh-Anh Phan <[email protected]> Co-authored-by: Harsha Nalluru <[email protected]> Co-authored-by: Wes Haggard <[email protected]> Co-authored-by: Sarangan Rajamanickam <[email protected]> Co-authored-by: Ben Broderick Phillips <[email protected]> Co-authored-by: Daniel Jurek <[email protected]> Co-authored-by: Timo van Veenendaal <[email protected]> Co-authored-by: ZiWei Chen <[email protected]> Co-authored-by: Scott Beddall (from Dev Box) <[email protected]> Co-authored-by: Richard Park <ripark\@microsoft.com> Co-authored-by: Jana R <[email protected]> Co-authored-by: Jeff Fisher <[email protected]> Co-authored-by: Mary Gao <[email protected]> Co-authored-by: Qiaoqiao Zhang <[email protected]> Co-authored-by: Glenn Harper <[email protected]> Co-authored-by: Deyaaeldeen Almahallawi <[email protected]> Co-authored-by: amariwest-msft <[email protected]> Co-authored-by: hamshavathimunibyraiah <[email protected]> Co-authored-by: Sharath <[email protected]> Co-authored-by: BrandonStudio <[email protected]> Co-authored-by: qiaozha <[email protected]> Co-authored-by: Driele Neves Ribeiro <[email protected]> Co-authored-by: Jesse Squire <[email protected]> Co-authored-by: Scott Addie <[email protected]> Co-authored-by: Ben Broderick Phillips <[email protected]> Co-authored-by: Manik Khandelwal <[email protected]> Co-authored-by: Manik Khandelwal <[email protected]> Co-authored-by: aditishree1 <[email protected]> Co-authored-by: Aditishree . <[email protected]> Co-authored-by: Albert Cheng <[email protected]> Co-authored-by: Aman Rao <[email protected]> Co-authored-by: koen-mertens <[email protected]> Co-authored-by: Karolien Gellynck <[email protected]> Co-authored-by: Karolien Gellynck <[email protected]> Co-authored-by: Karolien Gellynck <[email protected]> Co-authored-by: tomsft <[email protected]> Co-authored-by: kumarajay <[email protected]> Co-authored-by: Ryan Hurey <[email protected]> Co-authored-by: Hector Hernandez <[email protected]> Co-authored-by: Scott Beddall <[email protected]> Co-authored-by: rajuanitha88 <[email protected]>
…tion/release/beta4
### Describe the problem that is addressed by this PR Fixed streamDataParser audio data mapping ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary)
### Describe the problem that is addressed by this PR Fixed streamingDataParser import issue. ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary)
…re/azure-sdk-for-js into callautomation/release/beta4
### Packages impacted by this PR 1.3.0-beta.1 ### Issues associated with this PR Media streaming not added in the group call ### Describe the problem that is addressed by this PR Enable the media streaming for the group call ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary)
Upudated change log
…/azure-sdk-for-js into callautomation/release/ga4
Of the alpha packages. [Proving build](https://dev.azure.com/azure-sdk/internal/_build/results?buildId=4317759&view=results) Before my fix: <img width="754" alt="image" src="https://github.com/user-attachments/assets/bbb3653f-8ec2-4aed-9101-4bb519df5cbf"> After my fix: <img width="746" alt="image" src="https://github.com/user-attachments/assets/f1536bca-87c6-4ddd-bfa0-b9d90fd63b89">
Workaround to unblock JS pipeline failure when installing APIView parser.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Packages impacted by this PR
Issues associated with this PR
Describe the problem that is addressed by this PR
What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen?
Are there test cases added in this PR? (If not, why?)
Provide a list of related PRs (if any)
Command used to generate this PR:**(Applicable only to SDK release request PRs)
Checklists