Skip to content
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

Build fail: Cannot assign to "i" because it is a constant (@azure/communication-calling) #15479

Closed
2 tasks done
JamesBurnside opened this issue May 31, 2021 · 6 comments
Closed
2 tasks done
Assignees
Labels
Client This issue points to a problem in the data-plane of the library. Communication customer-reported Issues that are reported by GitHub users external to the Azure organization. needs-team-attention This issue needs attention from Azure service team or SDK team question The issue doesn't require a change to the product in order to be resolved. Most issues start as that

Comments

@JamesBurnside
Copy link
Member

(Created on behalf @canperk, original bug filed in samples repo: Azure-Samples/communication-services-web-calling-hero#64)

  • Package Name: @azure/communication-calling
  • Package Version: 1.0.0
  • Operating system: Win10
  • nodejs
    • version: v14.15.0
  • typescript
    • version: 4.3.2

Describe the bug
Build failure when using @azure/communication-calling in Vite/Vue app:

> node_modules/@azure/communication-calling/dist-esm/sdk.bundle.js:18:13900: error: Cannot assign to "i" because it is a constant

To Reproduce

  1. Setup Steps:

    > npm init @vitejs/app peerque --template vue-ts
    > cd peerque
    > npm i
    > npm i @azure/communication-calling@1.0.0 @azure/communication-common@1.0.0 @azure/communication-identity@1.0.0
    > npm run dev
  2. Repro steps:
    In main.ts add:

    import * as communication_calling from '@azure/communication-calling';
    console.log(communication_calling);

    Reload page.

  3. Result:

    11:12:15 a.m. [vite] new dependencies found: @azure/communication-calling, updating...
     > node_modules/@azure/communication-calling/dist-esm/sdk.bundle.js:18:13900: error: Cannot assign to "i" because it is a constant
        18 │ ...e),n=!1)},dispose:()=>{i=void 0}}}function asse...
           ╵                           ^
       node_modules/@azure/communication-calling/dist-esm/sdk.bundle.js:18:13765: note: "i" was declared a constant here
        18 │ ...r(){let e,t,n=!0;const i=new Promise(((n,i)=>{e...
           ╵                           ^

Expected behavior
No error

@ghost ghost added the needs-triage This is a new issue that needs to be triaged to the appropriate team. label May 31, 2021
@canperk
Copy link

canperk commented May 31, 2021

Thank you @JamesBurnside that you mentioned SDK repository that I couldn’t find on npm page.

@JamesBurnside
Copy link
Member Author

Thank you @JamesBurnside that you mentioned SDK repository that I couldn’t find on npm page.

You're right! Filed a separate issue for that to help folks in the future with filling issues and getting support #15480

@ramya-rao-a ramya-rao-a added Client This issue points to a problem in the data-plane of the library. Communication customer-reported Issues that are reported by GitHub users external to the Azure organization. needs-team-attention This issue needs attention from Azure service team or SDK team question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Jun 1, 2021
@ghost ghost removed the needs-triage This is a new issue that needs to be triaged to the appropriate team. label Jun 1, 2021
@DominikMe
Copy link
Member

Assigning to @akania who owns calling.

@DominikMe DominikMe assigned akania and unassigned beltr0n and DominikMe Jun 1, 2021
@akania
Copy link
Member

akania commented Jun 2, 2021

thank you for reporting, we'll check we've created ADO#2461772

@sharifrahaman
Copy link
Member

sharifrahaman commented Jun 18, 2021

We have found the root cause of this issue. It will be fixed in next release.

@sharifrahaman
Copy link
Member

sharifrahaman commented Jun 25, 2021

@akania akania closed this as completed Jun 28, 2021
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-js that referenced this issue Aug 20, 2021
Compute Rest API version 2021-07-01 (Azure#15479)
@github-actions github-actions bot locked and limited conversation to collaborators Apr 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Client This issue points to a problem in the data-plane of the library. Communication customer-reported Issues that are reported by GitHub users external to the Azure organization. needs-team-attention This issue needs attention from Azure service team or SDK team question The issue doesn't require a change to the product in order to be resolved. Most issues start as that
Projects
None yet
Development

No branches or pull requests

7 participants