MicrosoftDocs / msteams-docs

Source for the Microsoft Teams developer platform documentation.
https://aka.ms/teamsdev
Creative Commons Attribution 4.0 International
288 stars 512 forks source link

Teams app redeployment requires reinstallation #11849

Open lior-lew opened 2 days ago

lior-lew commented 2 days ago

Steps to reproduce

  1. I wrote a Teams app which is already published in MS App store.
  2. I deployed and publish same code under a different name to my organization only.
  3. I installed the app and it is working.
  4. I made changes in the typescript code. the changes was to add more logging.
  5. I deployed the app without republishing it.
  6. tried to send a notification and the notification did not reach my Teams chat.
  7. uninstalled and installed the app again
  8. everything is working

Expected behavior

when deploying new code that doesn't affect the manifest/permission/scope ... I expect that the application will work without the need to uninstall and install it again. after all, I did not republish it.

Actual behavior

after deploy of new Azure function, the notifications are not reaching my teams chat.

Error details

none. I am just getting an error that the member is not found.

      //findMember is expecting a function that returns a predicate. therefore, we must use async function
      //to return a promise that resolves to a predicate (even if the actual operation is synchronous)
      async (m) => (m.account.userPrincipalName.toLowerCase() === userEmail.toLowerCase() || 
                    m.account.email.toLowerCase() === userEmail.toLowerCase())
    ).catch((error) => {
      context.log.error(`Error finding member ${userEmail}:`, error);
    });
microsoft-github-policy-service[bot] commented 2 days ago

Hi lior-lew! Thank you for bringing this issue to our attention. We will investigate and if we require further information we will reach out in one business day. Please use this link to escalate if you don't get replies.

Best regards, Teams Platform

Meghana-MSFT commented 1 day ago

@lior-lew - This behavior seems to be expected. We will confirm this internally and get back to you.