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

fix wrongly replacing text in shared strings #81

Merged
merged 2 commits into from
Aug 9, 2018

Conversation

stevds
Copy link
Contributor

@stevds stevds commented Aug 9, 2018

This solves a similar problem as this pull request #69. Except this time it happens when there is only parts of strings that are substituted. For example, if you got something like "Page ${page}" in every sheet, in the output every "${page}" will have the same value.

@kant2002
Copy link
Collaborator

kant2002 commented Aug 9, 2018

@stevds Thanks for PR.
So essentially you want to preserve original substitution template, and let replacement engine do the thing on the next pages.
How difficult for you would be to add multi-page.xlsx so we could explicitly test your scenario ?

@kant2002 kant2002 merged commit 781bcab into optilude:master Aug 9, 2018
@kant2002
Copy link
Collaborator

@stevds shipped in 0.4.0

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.

2 participants