Merge branch 'main' into jaburnsi/support-existing-calls-adapters

.github/workflows/ci.yml

@@ -8,6 +8,7 @@ on:
       - release/**
   # Presubmit CI on PRs to all branches.
   pull_request:
+    types: [labeled, unlabeled, opened, synchronize, reopened]
   # Allows you to run this workflow manually from the Actions tab.
   workflow_dispatch:
 
@@ -67,7 +68,10 @@ jobs:
       # Checks-out your repository under $GITHUB_WORKSPACE so job can access it
       - uses: actions/checkout@v4
         with:
-          fetch-depth: 2 # this job requires depth of 2 for access to diff the SHA for change file check
+          # This job requires to fetch the entire history to access to diff the SHA for change file check.
+          # Otherwise, it can lead to issues when trying to compare branches, as the fetched commits may not
+          # include the common ancestor between the branches.
+          fetch-depth: 0
       # Ensure node version is great enough
       - name: Use Node.js
         uses: actions/setup-node@v4
@@ -86,10 +90,8 @@ jobs:
         if: ${{ always() && steps.install-dependencies.outcome == 'failure' }}
         run: echo "Failed to install dependencies, please make sure you run `rush update:all-flavors` if dependencies were updated." && exit 1
       - name: Check the necessary change file has been submitted
-        if: ${{ github.event_name == 'pull_request' && !contains( github.event.pull_request.labels.*.name, 'do not need changelog')}}
-        run: |
-          git fetch origin main:main --depth=2
-          node common/scripts/changelog/check.mjs ${{ github.event.pull_request.base.sha }} ${{ github.event.pull_request.head.sha }}
+        if: ${{ github.event_name == 'pull_request' && !contains( github.event.pull_request.labels.*.name, 'does not need changelog')}}
+        run: node common/scripts/changelog/check.mjs ${{ github.event.pull_request.base.sha }} ${{ github.event.pull_request.head.sha }}
 
       # Perform lint check
       - name: Run linter
@@ -144,8 +146,6 @@ jobs:
     steps:
       # Checks-out your repository under $GITHUB_WORKSPACE so job can access it
       - uses: actions/checkout@v4
-        with:
-          fetch-depth: 1
       # Ensure node version is great enough
       - name: Use Node.js
         uses: actions/setup-node@v4
@@ -239,8 +239,6 @@ jobs:
     steps:
       # Checks-out your repository under $GITHUB_WORKSPACE so job can access it
       - uses: actions/checkout@v4
-        with:
-          fetch-depth: 1
       # Ensure node version is great enough
       - name: Use Node.js
         uses: actions/setup-node@v4
@@ -293,8 +291,6 @@ jobs:
       matrix: ${{ fromJSON(needs.get_matrix.outputs.matrix) }}
     steps:
       - uses: actions/checkout@v4
-        with:
-          fetch-depth: 1
       - name: Use Node.js
         uses: actions/setup-node@v4
         with:
@@ -362,8 +358,6 @@ jobs:
       matrix: ${{ fromJSON(needs.get_matrix.outputs.matrix) }}
     steps:
       - uses: actions/checkout@v4
-        with:
-          fetch-depth: 1
       - name: Use Node.js
         uses: actions/setup-node@v4
         with:
@@ -431,8 +425,6 @@ jobs:
       matrix: ${{ fromJSON(needs.get_matrix.outputs.matrix) }}
     steps:
       - uses: actions/checkout@v4
-        with:
-          fetch-depth: 1
       - name: Use Node.js
         uses: actions/setup-node@v4
         with:
@@ -499,8 +491,6 @@ jobs:
       matrix: ${{ fromJSON(needs.get_matrix.outputs.matrix) }}
     steps:
       - uses: actions/checkout@v4
-        with:
-          fetch-depth: 1
       - name: Use Node.js
         uses: actions/setup-node@v4
         with:
@@ -555,8 +545,6 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-        with:
-          fetch-depth: 1
       - name: Use Node.js
         uses: actions/setup-node@v4
         with:
@@ -586,8 +574,6 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4
-        with:
-          fetch-depth: 1
       - name: Use Node.js
         uses: actions/setup-node@v4
         with:
@@ -619,8 +605,6 @@ jobs:
       matrix: ${{ fromJSON(needs.get_matrix.outputs.matrix) }}
     steps:
       - uses: actions/checkout@v4
-        with:
-          fetch-depth: 1
       - name: Use Node.js
         uses: actions/setup-node@v4
         with:
@@ -653,8 +637,6 @@ jobs:
       matrix: ${{ fromJSON(needs.get_matrix.outputs.matrix) }}
     steps:
       - uses: actions/checkout@v4
-        with:
-          fetch-depth: 1
       - name: Use Node.js
         uses: actions/setup-node@v4
         with:
@@ -687,8 +669,6 @@ jobs:
       matrix: ${{ fromJSON(needs.get_matrix.outputs.matrix) }}
     steps:
       - uses: actions/checkout@v4
-        with:
-          fetch-depth: 1
       - name: Use Node.js
         uses: actions/setup-node@v4
         with:
@@ -723,8 +703,6 @@ jobs:
       matrix: ${{ fromJSON(needs.get_matrix.outputs.matrix) }}
     steps:
       - uses: actions/checkout@v4
-        with:
-          fetch-depth: 1
       - name: Use Node.js
         uses: actions/setup-node@v4
         with:
@@ -775,8 +753,6 @@ jobs:
       matrix: ${{ fromJSON(needs.get_matrix.outputs.matrix) }}
     steps:
       - uses: actions/checkout@v4
-        with:
-          fetch-depth: 1
       - name: Use Node.js
         uses: actions/setup-node@v4
         with:

change-beta/@azure-communication-react-1aef335a-42b6-4070-9763-a0bc86f54ade.json

@@ -0,0 +1,9 @@
+{
+  "type": "prerelease",
+  "area": "fix",
+  "workstream": "js-loaders",
+  "comment": "Add new sub package to communication react to import the loader functions",
+  "packageName": "@azure/communication-react",
+  "email": "[email protected]",
+  "dependentChangeType": "patch"
+}

change/@azure-communication-react-28105d95-b06b-4e41-b4bb-ab47cf6cbbca.json → change-beta/@azure-communication-react-1df25fb6-0f83-436c-ab27-064736b43406.json

@@ -1,8 +1,8 @@
 {
   "type": "patch",
   "area": "improvement",
-  "workstream": "Calling dependency",
-  "comment": "Upgrade calling beta to 1.28.1-beta.1 and stable to 1.27.2",
+  "workstream": "Control button tooltip",
+  "comment": "Fix frequent tooltip dismissal when trying to mouse over tooltip content",
   "packageName": "@azure/communication-react",
   "email": "[email protected]",
   "dependentChangeType": "patch"

change/@azure-communication-react-adee31e2-62cb-4055-8a4d-851707056dc5.json → change-beta/@azure-communication-react-2370b346-59bb-4073-9237-5f3206c9dbc0.json

@@ -1,8 +1,8 @@
 {
-  "type": "patch",
+  "type": "minor",
   "area": "fix",
-  "workstream": "A11y Ezra",
-  "comment": "Fix visible names for pinning, camera and mic buttons",
+  "workstream": "Call Controls",
+  "comment": "Fix Call Controls to allow proper customization of buttons shown",
   "packageName": "@azure/communication-react",
   "email": "[email protected]",
   "dependentChangeType": "patch"

change-beta/@azure-communication-react-3609656d-b6a9-4253-93ee-50eacf1719e2.json

@@ -0,0 +1,9 @@
+{
+  "type": "patch",
+  "area": "fix",
+  "workstream": "AzureCommunicationChatAdapter",
+  "comment": "Adding logic to flag when the adapter is in the middle of creation",
+  "packageName": "@azure/communication-react",
+  "email": "[email protected]",
+  "dependentChangeType": "patch"
+}

change/@azure-communication-react-6389cbd7-eac4-44a1-bf98-7324b41c9932.json → change-beta/@azure-communication-react-4be604d9-5c4f-41fe-8fd4-cef3a57d8d48.json

@@ -1,8 +1,8 @@
 {
-  "type": "minor",
+  "type": "prerelease",
   "area": "feature",
-  "workstream": "Ad hoc calling",
-  "comment": "Stabilize, Inbound calling, PSTN, Teams Adhoc calling",
+  "workstream": "js-loaders",
+  "comment": "Move loaders to external directory import",
   "packageName": "@azure/communication-react",
   "email": "[email protected]",
   "dependentChangeType": "patch"

change/@azure-communication-react-7b73ca56-0fc7-4700-8367-c6276437787e.json → change-beta/@azure-communication-react-6d7a7ed6-36b4-4006-89de-49bf6ceacfec.json

@@ -1,8 +1,8 @@
 {
-  "type": "minor",
-  "area": "feature",
+  "type": "patch",
+  "area": "fix",
   "workstream": "DNS",
-  "comment": "Add DNS toggle to Microphone button menu options",
+  "comment": "Hide DNS on Safari",
   "packageName": "@azure/communication-react",
   "email": "[email protected]",
   "dependentChangeType": "patch"

change-beta/@azure-communication-react-8dad3734-cbd4-42d3-ae33-07faa73a4923.json

@@ -0,0 +1,9 @@
+{
+  "type": "patch",
+  "area": "fix",
+  "workstream": "CallSubscriber",
+  "comment": "Safely subscribe to on stateChanged events to prevent related events from failing if listeners throw an error",
+  "packageName": "@azure/communication-react",
+  "email": "[email protected]",
+  "dependentChangeType": "patch"
+}

change-beta/@azure-communication-react-a8b252ce-eb1d-4b71-b06f-b47055cea16b.json

@@ -0,0 +1,9 @@
+{
+  "type": "minor",
+  "area": "improvement",
+  "workstream": "Full screen captions",
+  "comment": "Caption fullscreen mode api",
+  "packageName": "@azure/communication-react",
+  "email": "[email protected]",
+  "dependentChangeType": "patch"
+}

change/@azure-communication-react-59e6959f-de97-409a-a1d1-20784e61652a.json → change-beta/@azure-communication-react-c2f1c492-b1a1-432b-b410-5fd852e81a29.json

@@ -1,8 +1,8 @@
 {
-  "type": "patch",
-  "area": "fix",
-  "workstream": "captions",
-  "comment": "Fix bug when adapter caption events are not firing",
+  "type": "minor",
+  "area": "feature",
+  "workstream": "Captions",
+  "comment": "GA acs captions",
   "packageName": "@azure/communication-react",
   "email": "[email protected]",
   "dependentChangeType": "patch"

change/@azure-communication-react-e89789b4-63cd-43fb-81ee-f0da59acf88a.json → change/@azure-communication-react-2370b346-59bb-4073-9237-5f3206c9dbc0.json

@@ -1,8 +1,8 @@
 {
-  "type": "patch",
+  "type": "minor",
   "area": "fix",
-  "workstream": "Ezra/a11y",
-  "comment": "Update contrast issues when interacting with other participants in the call",
+  "workstream": "Call Controls",
+  "comment": "Fix Call Controls to allow proper customization of buttons shown",
   "packageName": "@azure/communication-react",
   "email": "[email protected]",
   "dependentChangeType": "patch"

change/@azure-communication-react-3609656d-b6a9-4253-93ee-50eacf1719e2.json

@@ -0,0 +1,9 @@
+{
+  "type": "patch",
+  "area": "fix",
+  "workstream": "AzureCommunicationChatAdapter",
+  "comment": "Adding logic to flag when the adapter is in the middle of creation",
+  "packageName": "@azure/communication-react",
+  "email": "[email protected]",
+  "dependentChangeType": "patch"
+}