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

Enable library to work with Angular5 with @angular-cli #122

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

smkart
Copy link

@smkart smkart commented Oct 23, 2018

  • Changes to get latest code changes from SheeJS
  • Modified the way it acquired cptable
  • New approach works fine with @angular-cli

SheetJSDev and others added 12 commits December 31, 2016 03:20
- codepage updated to 1.6.0 (latest)
- clean up Makefile
- adapted .travis.yml to support 0.8
- removed test files that drifted from baseline
- removed XLSB pseudo-inverse tests due to fails in node 4+
- README and example cleanup
- basic XLSB and ODS write support
- flow typecheck for ODS file
  Note: xlsx.js flow fails: facebook/flow#380
- exposed jszip compression (fixes SheetJS#220, closes SheetJS#284)

README issues:

|  id  | author         | comment                                      |
|-----:|:---------------|:---------------------------------------------|
| SheetJS#202 | @sao93859      | closes SheetJS#202                                  |
| SheetJS#211 | @alexanderchan | closes SheetJS#211 corrected examples               |
| SheetJS#327 | @cskaandorp    | changed saveAs example to match write tests  |
| SheetJS#424 | @dskrvk        | added note about s2roa h/t @LeonardoPatignio |
| SheetJS#496 | @jimmywarting  | closes SheetJS#496 adapted rABS examples with rAAS  |

ODS file format issues:

|  id  | author         | comment                                      |
|-----:|:---------------|:---------------------------------------------|
| SheetJS#148 | @user4815162342| closes SheetJS#148 h/t @ziacik                      |
| SheetJS#166 | @paulproteus   | closes SheetJS#166 rudimentary ODS write support    |
| SheetJS#177 | @ziacik        | closes SheetJS#177                                  |
| SheetJS#179 | @ziacik        | closes SheetJS#179 use JSON when available          |
| SheetJS#317 | @ziacik        | closes SheetJS#317                                  |
| SheetJS#328 | @think01       | closes SheetJS#328                                  |
| SheetJS#383 | @mdamt         | closes SheetJS#383 duplicate cells should be copied |
| SheetJS#430 | @RB-Lab        | closes SheetJS#430                                  |
| SheetJS#546 | @lgodard       | closes SheetJS#546 thanks to other changes          |
- basic support for parsing BIFF2-4
- basic support for writing BIFF2
- cleaned up some bad substr uses for IE6 compatibility
- added flow type annotations for xlsx.flow.js
- added numerous null guards (fixes SheetJS#255 h/t @martinheidegger)
- README cleanup (fixes SheetJS#539 h/t @oliversalzburg)
- pin jszip to local version (closes SheetJS#408 h/t @limouri)

bower issues:

|  id  | author            | comment                                   |
|-----:|:------------------|:------------------------------------------|
| SheetJS#254 | @kkirsche         | fixes SheetJS#254 by removing version from json  |
| SheetJS#165 | @vincentcialdella | fixes SheetJS#165 by changing default script     |
| SheetJS#180 | @owencraig        | fixes SheetJS#180 by using xlsx.core.min.js      |

format issues:

|  id  | author            | comment                                   |
|-----:|:------------------|:------------------------------------------|
| SheetJS#271 | @morstaine        | fixes SheetJS#271 by reworking related parse fns |
| SheetJS#504 | @JanSchuermannPH  | fixes SheetJS#504 detect FullPaths h/t @Mithgol  |
| SheetJS#508 | @basma-emad       | fixes SheetJS#508 offending file used `x:` NS    |
SystemJS doesn't support require() , We need to use System.import to get script loading with SystermJS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants