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

Style xml file not present with xlsx file #521

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

Conversation

ak15
Copy link

@ak15 ak15 commented Nov 14, 2019

There is issue when a xlsx file does not contain style xml compressed with it,
we should create a blank style xml file so that it does not raise issue and stop processing xlsx file , only if the style xml is not present?
The below file was created with js library https://github.com/egeriis/zipcelx/wiki/How-to-use

Attached xlsx file created, which I was not able to upload
Main View_7oct14.xlsx

There can be other ways to solve this issue more efficiently but for me currently solved by making below changes

@coveralls
Copy link

coveralls commented Nov 14, 2019

Coverage Status

Coverage increased (+0.03%) to 94.4% when pulling a8ddffc on ak15:master into 4ec1104 on roo-rb:master.

@ak15
Copy link
Author

ak15 commented Nov 14, 2019

@chopraanmol1 it would be great if you could look into this issue and review the code, because it was breaking for me

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.

None yet

2 participants