sheetjs-style icon indicating copy to clipboard operation
sheetjs-style copied to clipboard

Fix 4 variable errors in xlsx.js

Open Mycornia opened this issue 2 years ago • 5 comments

Fixes the following undefined variables in xlsx.js:

  • QUOTE at line 21524
  • _hashIndex at line 21603
  • _listIndex at line 21604
  • style_builder at line 20830

Fixes the following (closed) issues: #7 & #13

I know this repository has not been updated since November 2021, however creating a PR seems like the right thing to do.

Mycornia avatar Jul 14 '23 07:07 Mycornia

Unless this merges, the app will crash.

MubashirullahD avatar Jul 30 '23 06:07 MubashirullahD

merging this pr will close this issue #17. A quick fix is to create a patch with your fixes within your project

clefayomide avatar Sep 14 '23 15:09 clefayomide

OMG how is this still open?

jbaccarelli-equinix avatar Apr 15 '24 13:04 jbaccarelli-equinix

OMG how is this still open?

Sadly the repo has been dead since late 2021, however there’s I’ve forked the repo so you can use that

Mycornia avatar Apr 15 '24 18:04 Mycornia

OMG how is this still open?

Sadly the repo has been dead since late 2021, however there’s I’ve forked the repo so you can use that

hah ok thanks for the quick reply :) i managed to get my code to work using the xlsx package, so no worries.

jbaccarelli-equinix avatar Apr 16 '24 08:04 jbaccarelli-equinix