Type 'undefined' is not assignable to type 'T'. about parchment HOT 7 CLOSED

@danielmhair Same boat as you, only error present.

from parchment.

I'm getting this also when I installed ngx-quill, which installs quill and @types/quill. It only happens when my tsconfig.json file has strict: true. Setting it to strict: false fixes this. However, I need strict: true on my project. So I made a pull request. But to explain in detail, my error is:

node_modules/@types/quill/node_modules/quill/parchment/src/collection/linked-list.ts(9,5)

For this PR, I had to change all undefined to null. All tests are passing. Please let me know if this is not what you are expecting with the linked list. I understand that setting it to null instead of undefined could cause unintended issues, but since its passing tests, I'm assuming the unit tests are covering our bases?

Here is the pull request: #43

from parchment.

Hey @jhchen, after you merged master, I tried updating my version of parchment, but it seems to not receive the updates. The version that it installs is 1.1.3. I saw that master has the same version number. Have the changes not been published to npm yet?

from parchment.

@danielmhair seeing the same issue as you.

from parchment.

@danielmhair I was going through the same thing and was about to submit a PR as well, seems it is important to allow developers using strict: true to use the types free of error. Beat me to it!

from parchment.

Sounds great @juanparadox! To fix it, I set strict: true initially, and there are a ton more errors that come up from, but since this was the only error that comes up on my end, I felt no need to do more of it. I have tested these changes in my project and it does seem to fix the build errors.

from parchment.

No it's not pushed. There's some more diligence I need to do before publishing so I'll try to get to that this weekend. FYI for any project that uses version numbers you can look in the commit log and check if the change you are looking for is before or after the version bump.

from parchment.