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

TypeError: range_1.default is not a constructor #220

Open
Jedliu opened this issue Dec 2, 2022 · 5 comments
Open

TypeError: range_1.default is not a constructor #220

Jedliu opened this issue Dec 2, 2022 · 5 comments

Comments

@Jedliu
Copy link

Jedliu commented Dec 2, 2022

Hi @Siemienik , I'm trying to include this in the nx project and do the tests to add image insertion. It's using Jest for testing and there is an error

TypeError: range_1.default is not a constructor

in xlsx-renderer/src/Scope.ts so the tests failed.

I tried to include the Range def in index.d.ts of exceljs package but it didn't work.

Do you have a clue how to fix this? Thank you very much!

图片

@Siemienik
Copy link
Owner

Are you able to link the repo?

@Jedliu
Copy link
Author

Jedliu commented Dec 5, 2022

Hi @Siemienik, thanks for your fast reply. I will create a repo for your reference soon.

@Jedliu
Copy link
Author

Jedliu commented Dec 7, 2022

I solved this by forking exceljs and do a patch then install from this patch. Thanks for your attention.
Here is a link to the patch release

PS: exceljs hasn't been updated for quite a long time.

@Jedliu Jedliu closed this as completed Dec 7, 2022
@Siemienik
Copy link
Owner

@Jedliu, Thanks for info.

Recently I started to organize Merge Fest for my projects, including exceljs. The MergeFests are organized and announced on our Discord: https://discord.gg/siema . Today I will do next one at 8pm CEST. Join us if you would like. Especially if have you got some propositions which PRs are more important than rest, just send me a link on #exceljs channel.

As I can't release exceljs, I started to release it as @siema-team/spreadsheets, GH: https://github.com/SiemaTeam/spreadsheets

Would you mind send me a link to PR with your fixation? or open new PR if doesn't exist?

Regards

@Jedliu
Copy link
Author

Jedliu commented Apr 25, 2023

Hi, Siemienik. Thanks for getting in touch with me. I will look into this when I have spare time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

2 participants