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
deep require is no longer supported for uuid library #70
Comments
@gregdolley Would you mind checking this out? |
Will do!
…On Tue, Jan 12, 2021, 09:59 Mark Daggett ***@***.***> wrote:
@gregdolley <https://github.com/gregdolley> Would you mind checking this
out?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#70 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABN3O6L7EO5BRN64RSHLFQDSZSEYVANCNFSM4UFDGIVA>
.
|
@heavysixer: this is popping up because we use "require" the uuid library like this in many places: const uuidv4 = require('uuid/v4'); ...where there is a "uuid/" inside the require param. The warning is telling us, that in uuid lib v7.x (we are on 3.x), we can't import it like this anymore, but rather should use the "import" syntax or something like: "const { v4: uuidv4 } = require('uuid');" for commonJS. But since we're still on v3.x, we shouldn't have to worry about this. I'll try and find why @hannaol was still getting this error. |
@hannaol - what version of node are you running your code under? Also, could you please post the part of your code that is causing this error? Thanks! |
The issue is still present and makes the library unusable |
Sorry, i totally forgot about replying to this! So for me the issue was caused because another library in my project was also using uuid, but version 8. As far as i can remember i installed that dependency before adding nodejs-pptx. (node version 12, and the code that caused this error to arise was the code sample to create a new powerpoint presentation, as far as i can remember) |
First of all, thank you for your very helpful library !
I ran into the following problem:
In the file node-pptx/lib/factories/ppt/index.js line 5 a deep requires is used for the uuid library :
const uuidv4 = require('uuid/v4');
According to their page this is deprecated and no longer supported: https://github.com/uuidjs/uuid#deep-requires-no-longer-supported
Therefore the following error is thrown in my project: Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: Package subpath './v4' is not defined by "exports" in .../node_modules/uuid/package.json
The text was updated successfully, but these errors were encountered: