Skip to content

Commit b3e4395

Browse files
authored
Merge pull request #231080 from garchiro7/main
ACS Calling SDK fixes
2 parents ade717e + b5f88d0 commit b3e4395

27 files changed

+373
-326
lines changed

articles/communication-services/concepts/developer-tools/network-diagnostic.md

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ The **Network Diagnostics Tool** enables Azure Communication Services developers
2121

2222
As part of the diagnostics performed, the user is asked to enable permissions for the tool to access their devices. Next, the user is asked to record their voice, which is then played back using an echo bot to ensure that the microphone is working. The tool finally, performs a video test. The test uses the camera to detect video and measure the quality for sent and received frames.
2323

24-
If you are looking to build your own Network Diagnostic Tool or to perform deeper integration of this tool into your application, you can leverage [pre-call diagnostic APIs](../voice-video-calling/pre-call-diagnostics.md) for the calling SDK.
24+
If you're looking to build your own Network Diagnostic Tool or to perform deeper integration of this tool into your application, you can leverage [pre-call diagnostic APIs](../voice-video-calling/pre-call-diagnostics.md) for the calling SDK.
2525

2626
## Performed tests
2727

@@ -36,11 +36,13 @@ If you are looking to build your own Network Diagnostic Tool or to perform deepe
3636

3737
## Privacy
3838

39-
When a user runs a network diagnostic, the tool collects and store service and client telemetry data to verify your network conditions and ensure that they're compatible with Azure Communication Services. The telemetry collected doesn't contain personal identifiable information. The test utilizes both audio and video collected through your device for this verification. The audio and video used for the test aren't stored.
39+
When a user runs a network diagnostic, the tool collects and store service and client telemetry data to verify your network conditions and ensure that they're compatible with Azure Communication Services.
40+
41+
The telemetry collected doesn't contain personal identifiable information. The test utilizes both audio and video collected through your device for this verification. The audio and video used for the test aren't stored.
4042

4143
## Support
4244

43-
The test provides a **unique identifier** for your test which you can provide our support team who can provide further help. For more information see [help and support options](../../support.md)
45+
The test provides a **unique identifier** for your test, which you can provide our support team who can provide further help. For more information, see [help and support options](../../support.md)
4446

4547
## Next Steps
4648

articles/communication-services/concepts/router/classification-concepts.md

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ ms.service: azure-communication-services
1818

1919
When you submit a job to Job Router, you can either specify the queue, priority, and worker selectors manually or you can specify a classification policy to drive these values.
2020

21-
If you choose to use a classification policy, you will receive a [JobClassified Event][job_classified_event] or a [JobClassificationFailed Event][job_classify_failed_event] with the result. Once the job has been successfully classified, it will be automatically queued. If the classification process fails, you'll need to intervene to fix it.
21+
If you choose to use a classification policy, you receive a [JobClassified Event][job_classified_event] or a [JobClassificationFailed Event][job_classify_failed_event] with the result. Once the job has been successfully classified, it's automatically queued. If the classification process fails, you need to intervene to fix it.
2222

2323
The process of classifying a Job involves optionally setting the following properties:
2424

@@ -30,41 +30,41 @@ The process of classifying a Job involves optionally setting the following prope
3030

3131
The priority of a Job can be resolved during classification using one of many rule engines.
3232

33-
See the [Rule concepts](router-rule-concepts.md) page for more information.
33+
For more information, see the [Rule concepts](router-rule-concepts.md) page.
3434

3535
## Worker selectors
3636

37-
Each job carries a collection of worker selectors, that are evaluated against the worker labels. These are conditions that need to be true of a worker to be a match.
38-
You can use the classification policy to attach these conditions to a job. You can do this by specifying one or more selector attachments.
37+
Each job carries a collection of worker selectors that's evaluated against the worker labels. These conditions need to be true of a worker to be a match.
38+
You can use the classification policy to attach these conditions to a job; you can do it by specifying one or more selector attachments.
3939

40-
For more information see the section [below](#using-label-selector-attachments).
40+
For more information, see, the section: [using label selector attachments](#using-label-selector-attachments).
4141

4242
## Queue selectors
4343

4444
You can also specify a collection of label selector attachments to select the Queue based on its labels.
4545

46-
For more information see the section [below](#using-label-selector-attachments).
46+
For more information, see, the section: [using label selector attachments](#using-label-selector-attachments).
4747

4848
## Using label selector attachments
4949

5050
The following label selector attachments are available:
5151

5252
**Static label selector -** Always attaches the given `LabelSelector` to the Job.
5353

54-
**Conditional label selector -** Evaluates a condition defined by a [rule](router-rule-concepts.md). If it resolves to `true`, then the specified collection of selectors will be attached to the Job.
54+
**Conditional label selector -** Evaluates a condition defined by a [rule](router-rule-concepts.md). If it resolves to `true`, then the specified collection of selectors are attached to the Job.
5555

5656
**Passthrough label selector -** Attaches a selector to the Job with the specified key and operator but gets the value from the Job label of the same key.
5757

58-
**Rule label selector -** Sources a collection of selectors from one of many rule engines. Read the [RouterRule concepts](router-rule-concepts.md) page for more information.
58+
**Rule label selector -** Sources a collection of selectors from one of many rule engines. Read the [RouterRule concepts](router-rule-concepts.md) page, for more information.
5959

6060
**Weighted allocation label selector -** Enables you to specify a percentage-based weighting and a collection of selectors to attach based on the weighting allocation. For example, you may want 30% of the Jobs to go to "Vendor 1" and 70% of Jobs to go to "Vendor 2".
6161

6262
## Reclassifying a job
6363

6464
Once a Job has been classified, it can be reclassified in the following ways:
6565

66-
1. You can update the Job labels, which will cause the Job Router to evaluate the new labels with the previous Classification Policy.
67-
2. You can update the Classification Policy ID of a Job, which will cause Job Router to process the existing Job against the new policy.
66+
1. You can update the Job labels, which cause the Job Router to evaluate the new labels with the previous Classification Policy.
67+
2. You can update the Classification Policy ID of a Job, which causes Job Router to process the existing Job against the new policy.
6868
3. An Exception Policy **trigger** can take the **action** of requesting a Job be reclassified.
6969

7070
> [!NOTE]

articles/communication-services/concepts/ui-library/includes/mobile-ui-use-cases.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ The `ChatComposite` was designed to fit into different layouts and views in your
101101

102102
## Supported identities
103103

104-
To initialize a composite and authenticate to the service, a user must have an Azure Communication Services identity. For more information, see [Authenticate to Azure Communication Services](../../authentication.md) and [Quickstart: Create and manage access tokens](../../../quickstarts/identity/access-tokens.md).
104+
To initialize a composite, and authenticate to the service, a user must have an Azure Communication Services identity. For more information, see [Authenticate to Azure Communication Services](../../authentication.md) and [Quickstart: Create and manage access tokens](../../../quickstarts/identity/access-tokens.md).
105105

106106
## Recommended architecture
107107

@@ -129,7 +129,7 @@ Call and chat client libraries must have the context for the call they join. Lik
129129

130130
When troubleshooting happens for voice or video calls, you may be asked to provide a **CallID**; this ID is used to identify Communication Services calls.
131131

132-
This CallID can be retrieved via the action bar on the bottom of the call screen; you'll see an ellipsis button; once the user performs the tap action an option of ***"Share diagnostics info"***; the user will have the capability to share **the diagnostics info** that will be required to track any issues by the support team.
132+
This CallID can be retrieved via the action bar on the bottom of the call screen; you see an ellipsis button; once the user performs the tap action an option of ***"Share diagnostics info"***; the user can share **the diagnostics info** that's required to track any issues by the support team.
133133

134134
For programmatic access to **CallID**, see ["How to get debug information programmatically"](../../../how-tos/ui-library-sdk/troubleshooting.md).
135135

articles/communication-services/concepts/ui-library/includes/web-ui-library.md

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -37,20 +37,20 @@ Developers can easily instantiate the Composite using an Azure Communication Ser
3737

3838
| Composite | Use Cases |
3939
| ------------ | ---------- |
40-
| [CallwithChatComposite](https://azure.github.io/communication-ui-library/?path=/docs/composites-call-with-chat-basicexample--basic-example) | Experience combining calling and chat features to allow users to start or join a call and chat thread. In the experience the user has the ability to both communicate using voice and video, as well as access to a rich chat thread where messages can be exchanged between participants. It includes support for Teams Interop. |
41-
| [CallComposite](https://azure.github.io/communication-ui-library/?path=/docs/composites-call-basicexample--basic-example) | Calling experience that allows users to start or join a call. Inside the experience users can configure their devices, participate in the call with video, and see other participants, including those participants with video turn on. For Teams Interop is includes lobby functionality for user to wait to be admitted. |
40+
| [CallwithChatComposite](https://azure.github.io/communication-ui-library/?path=/docs/composites-call-with-chat-basicexample--basic-example) | Experience combining calling and chat features to allow users to start or join a call and chat thread. In the experience, the user has the ability to both communicate using voice and video, and access to a rich chat thread where messages can be exchanged between participants. It includes support for Teams Interop. |
41+
| [CallComposite](https://azure.github.io/communication-ui-library/?path=/docs/composites-call-basicexample--basic-example) | Calling experience that allows users to start or join a call. Inside the experience users can configure their devices, participate in the call with video, and see other participants, including those participants with video turn-on. For Teams Interop, is included lobby functionality for user to wait to be admitted. |
4242
| [ChatComposite](https://azure.github.io/communication-ui-library/?path=/docs/composites-chat-basicexample--basic-example) | Chat experience where user can send and receive messages. Thread events like typing, reads, participants entering and leaving are displayed to the user as part of the chat thread. |
4343
## UI Component overview
4444

45-
Pure UI Components that can be used by developers to compose communication experiences, from stitching video tiles into a grid to showcase remote participants, to organizing components to fit your applications specifications.
45+
Pure UI Components can be used for the developers, to compose communication experiences, from stitching video tiles into a grid to showcase remote participants, to organizing components to fit your applications specifications.
4646
UI Components support customization to give the components the right feel and look to match an applications branding and style.
4747

4848
| Area | Component | Description |
4949
| ------- | ------------ | ----------------- |
5050
| Calling | [Grid Layout](https://azure.github.io/communication-ui-library/?path=/story/ui-components-gridlayout--grid-layout) | Grid component to organize Video Tiles into an NxN grid |
5151
| | [Video Tile](https://azure.github.io/communication-ui-library/?path=/story/ui-components-videotile--video-tile) | Component that displays video stream when available and a default static component when not |
5252
| | [Control Bar](https://azure.github.io/communication-ui-library/?path=/story/ui-components-controlbar--control-bar) | Container to organize DefaultButtons to hook up to specific call actions like mute or share screen |
53-
| | [VideoGallery](https://azure.github.io/communication-ui-library/?path=/story/ui-components-video-gallery--video-gallery) | Turn-key video gallery component which dynamically changes as participants are added |
53+
| | [VideoGallery](https://azure.github.io/communication-ui-library/?path=/story/ui-components-video-gallery--video-gallery) | Turn-key video gallery component, which dynamically changes as participants are added |
5454
| | [Dialpad](https://azure.github.io/communication-ui-library/?path=/docs/ui-components-dialpad--dialpad) | Component to support phone number input and DTMF tones. |
5555
| Chat | [Message Thread](https://azure.github.io/communication-ui-library/?path=/story/ui-components-messagethread--message-thread) | Container that renders chat messages, system messages, and custom messages |
5656
| | [Send Box](https://azure.github.io/communication-ui-library/?path=/story/ui-components-sendbox--send-box) | Text input component with a discrete send button |
@@ -61,20 +61,20 @@ UI Components support customization to give the components the right feel and lo
6161

6262
## Installing Web UI Library
6363

64-
Stateful clients are found as part of the `@azure/communication-react` package.
64+
Stateful clients are found as part of the `@azure/communication-react` package.
6565

6666
```bash
6767
npm i --save @azure/communication-react
6868
```
6969

7070
## What UI artifact is best for my project?
7171

72-
Understanding these requirements will help you choose the right client library:
72+
Understanding these requirements help you choose the right client library:
7373

7474
- **How much customization do you desire?** Azure Communication core client libraries don't have a UX and are designed so you can build whatever UX you want. UI Library components provide UI assets at the cost of reduced customization.
7575
- **What platforms are you targeting?** Different platforms have different capabilities.
7676

77-
Details about feature availability in the [UI Library is available here](https://azure.github.io/communication-ui-library/?path=/story/use-cases--page), but key trade-offs are summarized below.
77+
Details about feature availability in the [UI Library is available here](https://azure.github.io/communication-ui-library/?path=/story/use-cases--page), but key trade-offs are summarized in the next table.
7878

7979
| Client library / SDK | Implementation Complexity | Customization Ability | Calling | Chat | [Teams Interop](../../teams-interop.md) |
8080
| --------------------- | ------------------------- | --------------------- | ------- | ---- | ----------------------------------------------------------------------------------------------------- |

articles/communication-services/concepts/ui-library/includes/web-ui-use-cases.md

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,8 @@ Get more conceptual documentation, quickstarts, and examples in the [UI Library
5353

5454
## Supported identities
5555

56-
To initialize a composite and authenticate to the service, a user must have an Azure Communication Services identity. For more information, see [Authenticate to Azure Communication Services](../../authentication.md) and [Quickstart: Create and manage access tokens](../../../quickstarts/identity/access-tokens.md).
56+
To initialize a composite, and authenticate to the service, a user must have an Azure Communication Services identity. For more information, see [Authenticate to Azure Communication Services](../../authentication.md) and [Quickstart: Create and manage access tokens](../../../quickstarts/identity/access-tokens.md).
57+
5758

5859
## Teams interoperability
5960

@@ -99,7 +100,7 @@ The following table compares composites and UI components for observability use
99100

100101
## Recommended architecture
101102

102-
Initialize a composite and base components by using an Azure Communication Services access token. It's important to get access tokens from Communication Services through a trusted service that you manage. For more information, see [Quickstart: Create and manage access tokens](../../../quickstarts/identity/access-tokens.md) and the [trusted service tutorial](../../../tutorials/trusted-service-tutorial.md).
103+
Initialize a composite and base component by using an Azure Communication Services access token. It's important to get access tokens from Communication Services through a trusted service that you manage. For more information, see [Quickstart: Create and manage access tokens](../../../quickstarts/identity/access-tokens.md) and the [trusted service tutorial](../../../tutorials/trusted-service-tutorial.md).
103104

104105
:::image type="content" source="../../media/mobile-ui/ui-library-architecture.png" border="false" alt-text="Diagram that shows the recommended UI Library architecture.":::
105106

@@ -122,7 +123,7 @@ The following table summarizes initialization and resource management functions
122123

123124
\* The current version of Chrome and the two preceding releases are supported.
124125

125-
\*\* Safari version 13.1 and later versions are supported. Outgoing video for Safari macOS isn't yet supported, but it is supported for iOS. Outgoing screen sharing is supported only on desktop iOS.
126+
\*\* Safari version 13.1 and later versions are supported. Outgoing video for Safari macOS isn't yet supported, but it's supported for iOS. Outgoing screen sharing is supported only on desktop iOS.
126127

127128
## Accessibility
128129

0 commit comments

Comments
 (0)