Skip to content

Added Images cell data type

Miniontobyrequested to merge
Miniontoby/write-excel-file:main into main

Implemented the protocol to include images in spreadsheets

Stuff I did:

  • Written the code for adding the images
  • Rewritten your code so it would bring back the images list to the write javascript file
  • Added the code to include the newly created xml files in both NodeJS and browser side
  • Added copyFile async function wrapper for when a string is passed in as data to be used as path. fs.copyFile is supported since Node v9. There is a ponyfill available if you really want to support Node v8 and below.
  • Enabled the typescript typings you already added but which you commented out
  • Added some extra typescript documentation strings to the value object of the Image type
  • Tested both NodeJS and browser to see if they worked
  • I made the filename required, since with that you then specify the extension of the image as well. In my opinion it made more sense to add it, since else I would need to give the files random (or incremental) names instead... which of course isn't a problem so please give your opinion on this!

Only misses test file(s).

Merge request reports