Skip to content
This repository was archived by the owner on May 26, 2022. It is now read-only.

Conversation

@adrilo
Copy link
Collaborator

@adrilo adrilo commented May 18, 2016

Fixes #207

Instead of relying on the ID, sheets should be retrieved in the order they appear in the file.
Workbook.xml describes the correct order.
This allows the reader to read data in the correct order when sheets have been manually moved after creation.

@boxcla
Copy link

boxcla commented May 18, 2016

Verified that @adrilo has signed the CLA. Thanks for the pull request!

@adrilo adrilo force-pushed the fix_sheet_ordering_with_xlsx_files branch 8 times, most recently from de632fb to ea00024 Compare May 19, 2016 17:19
Instead of relying on the ID, sheets should be retrieved in the order they appear in the file.
Workbook.xml describes the correct order.
This allows the reader to read data in the correct order when sheets have been manually moved after creation.
@adrilo adrilo merged commit b8fd789 into master May 19, 2016
@adrilo adrilo deleted the fix_sheet_ordering_with_xlsx_files branch May 19, 2016 17:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants