Skip to content

Added Documentaion for issue #2139#2210

Closed
himanshusaini111 wants to merge 1 commit intoSheetJS:masterfrom
himanshusaini111:added_doc_for_mutated_header
Closed

Added Documentaion for issue #2139#2210
himanshusaini111 wants to merge 1 commit intoSheetJS:masterfrom
himanshusaini111:added_doc_for_mutated_header

Conversation

@himanshusaini111
Copy link
Contributor

Issue: #2139
Added Documentation to specify the mutable nature of headers passed and why it's intended and actually useful.

@SheetJSDev
Copy link
Contributor

Blurb should be moved below the table. See https://github.com/SheetJS/sheetjs/blob/master/README.md#parsing-options as an example of how the notes should be laid out (** should be in the description for header).

@paustint thoughts?

@himanshusaini111
Copy link
Contributor Author

himanshusaini111 commented Jan 6, 2021

Blurb should be moved below the table. See https://github.com/SheetJS/sheetjs/blob/master/README.md#parsing-options as an example of how the notes should be laid out (** should be in the description for header).

@paustint thoughts?

Thanks for the feedback.
Should I push another commit on this branch only or raise another pr with another branch?
@SheetJSDev
@paustint

Update: I have updated this PR please check @SheetJSDev @paustint

moved the note below the table
@himanshusaini111 himanshusaini111 force-pushed the added_doc_for_mutated_header branch from 34a1bcd to 09396c0 Compare January 6, 2021 05:59
@SheetJSDev SheetJSDev force-pushed the master branch 2 times, most recently from 7d51b80 to 1d7aff4 Compare April 13, 2021 20:06
@reviewher
Copy link
Contributor

af34ae4

@reviewher reviewher closed this Sep 11, 2021
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