Company Communicator v4.1.1 (#423)

- Update version number in manifests to 4.1.1
- Fix bug in #420 where it tries to get the member information for the bot, resulting for 404 Not Found errors
- Ensure Azure tables are created by prep function. This allows a fix for the issue reported in #420 by simply deleting the UserData table and restarting the prep Azure function.
- Update npm packages

Deployment/deploy.ps1

@@ -926,7 +926,7 @@ function logout {
 
 # Initialize connections - Azure Az/CLI/Azure AD
     WriteI -message "Login with with your Azure subscription account. Launching Azure sign-in window..."
-    Connect-AzAccount -Subscription $parameters.subscriptionId.Value -ErrorAction Stop
+    Connect-AzAccount -Subscription $parameters.subscriptionId.Value -Tenant $parameters.subscriptionTenantId.value -ErrorAction Stop
     $user = az login --tenant $parameters.subscriptionTenantId.value
     if ($LASTEXITCODE -ne 0) {
         WriteE -message "Login failed for user..."

Manifest/manifest_authors.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://developer.microsoft.com/en-us/json-schemas/teams/v1.5/MicrosoftTeams.schema.json",
   "manifestVersion": "1.5",
-  "version": "4.1.0",
+  "version": "4.1.1",
   "id": "1c07cd26-a088-4db8-8928-ace382fa219f",
   "packageName": "com.microsoft.teams.companycommunicator.authors",
   "developer": {

Manifest/manifest_users.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://developer.microsoft.com/en-us/json-schemas/teams/v1.5/MicrosoftTeams.schema.json",
   "manifestVersion": "1.5",
-  "version": "4.1.0",
+  "version": "4.1.1",
   "id": "148a66bb-e83d-425a-927d-09f4299a9274",
   "packageName": "com.microsoft.teams.companycommunicator",
   "developer": {

Source/CompanyCommunicator.Prep.Func/Startup.cs

@@ -57,7 +57,7 @@ public override void Configure(IFunctionsHostBuilder builder)
                     // tables exist. It is here as a possible configuration setting in
                     // case it needs to be set differently.
                     repositoryOptions.EnsureTableExists =
-                        !configuration.GetValue<bool>("IsItExpectedThatTableAlreadyExists", true);
+                        !configuration.GetValue<bool>("IsItExpectedThatTableAlreadyExists", false);
                 });
             builder.Services.AddOptions<MessageQueueOptions>()
                 .Configure<IConfiguration>((messageQueueOptions, configuration) =>

Source/CompanyCommunicator/Bot/TeamsDataCapture.cs

@@ -69,8 +69,8 @@ public async Task OnBotAddedAsync(ITurnContext turnContext, IConversationUpdateA
                     await this.teamDataRepository.SaveTeamDataAsync(activity);
                     break;
                 case TeamsDataCapture.PersonalType:
-                    // Skip Guest users.
-                    TeamsChannelAccount teamsUser = await TeamsInfo.GetMemberAsync(turnContext, activity.Recipient.Id, cancellationToken);
+                    // Skip Guest users
+                    TeamsChannelAccount teamsUser = await TeamsInfo.GetMemberAsync(turnContext, activity.From.Id, cancellationToken);
                     if (!teamsUser.UserPrincipalName.ToLower().Contains("#ext#"))
                     {
                         await this.userDataService.SaveUserDataAsync(activity);