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

fixed a bug related to not escaping newlines in text fields #25

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

icodeforlove
Copy link

Theres currently a bug where if you try to copy a field that has newlines the output will not escape them. This fixes that.

@icodeforlove
Copy link
Author

icodeforlove commented Nov 1, 2016

Just noticed #18, please merge that fix in as it resolves the need to merge this one.

This was not the case there were still issues related to \r\n. But both tickets demonstrate the bug, which should be fixed.

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.

1 participant