-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Angular is consuming dist-esm instead of dist #7296
Comments
Thanks for reporting the issue, @johnbigley . The issue is on our radar, @jeremymeng is coming up with the fix soon. |
Thank you @johnbigley! Could you please provide a minimal project or steps to do to reproduce this issue? We think we know where the problem is but would like to confirm that it is actually the issue you are hitting. |
Unfortunately, I wrote a proof of concept and it worked fine. I put it in our product and I ran into this error. I am not sure what the difference is. I can't give you our product, but I will try to reproduce it. |
I am unable to reproduce this outside of the product code. Any progress? |
@johnbigley we merged in a fix in PR #7298. Would you be able to give the private build a try? The storage-blob package can be download from https://dev.azure.com/azure-sdk/public/_build/results?buildId=255573&view=artifacts&type=publishedArtifacts Once you download the file |
Yes, I will give it a try and get back to you. |
I am getting the following compilation errors: |
@johnbigley what version of TypeScript are you using? |
"typescript": "~3.1.1" |
@johnbigley We migrated to typescript 3.7 for building our libraries and the generated type definition file are not compatible with typescript 3.1~3.5. @xirzec submit a fix to also include types that are compatible with down-level typescript compilers. Could you please try out the private build from https://dev.azure.com/azure-sdk/public/_build/results?buildId=258631&view=artifacts&type=publishedArtifacts? The file to download is |
I am now getting this runtime error: |
@johnbigley this is mysterious. Does this only happen in your production? If not, could you please share a repro project? |
It happens in production. I will be looking into it today. |
It was a problem with some new html. It now compiles and runs. When will this version be released? |
Glad to know that! We just released the new version today. |
I appreciate you addressing this problem so quickly. It makes a big impact on the project that I am working on. |
Closing as the issue is fixed. |
When I navigate to the Angular App I see the following in the Chrome console:
utils.node.js:159 Uncaught TypeError: util__WEBPACK_IMPORTED_MODULE_2__.promisify is not a function
at Module../node_modules/@azure/storage-blob/dist-esm/src/utils/utils.node.js (utils.node.js:159)
at webpack_require (bootstrap:78)
at Module../node_modules/@azure/storage-blob/dist-esm/src/Clients.js (Clients.js:1)
at webpack_require (bootstrap:78)
at Module../node_modules/@azure/storage-blob/dist-esm/src/BlobServiceClient.js (BlobServiceClient.js:1)
at webpack_require (bootstrap:78)
at Module../node_modules/@azure/storage-blob/dist-esm/src/index.browser.js (index.browser.js:1)
at webpack_require (bootstrap:78)
at Module../src/app/common/services/blob-storage-credentials.service.ts (user.model.ts:9)
at webpack_require (bootstrap:78)
It appears that it is loading the files from the node.js code instead of the browser code.
I installed this using npm install @azure/storage-blob. I don't know if this is a bug or a configuration error on my part. Any guidance would be appreciated.
The text was updated successfully, but these errors were encountered: