Skip to content

Added Option and Testing For KeepExistingFields - #54

Open
whitef0x0 wants to merge 19 commits into
pdffillerjs:masterfrom
whitef0x0:modifications
Open

Added Option and Testing For KeepExistingFields#54
whitef0x0 wants to merge 19 commits into
pdffillerjs:masterfrom
whitef0x0:modifications

Conversation

@whitef0x0

Copy link
Copy Markdown
Contributor

Added option and relevant tests for the new KeepExistingFields functionality.

@whitef0x0 whitef0x0 mentioned this pull request Feb 27, 2017
@wmbutler

Copy link
Copy Markdown

Trying to pass in as an option but your module.exports is not a function.

@wmbutler

Copy link
Copy Markdown

@whitef0x0 I don't think this works. I've tested it a few times with the setOptions and it's failing to show fields.

@wmbutler

Copy link
Copy Markdown

ok, I tried. I'm going to fork this myself under another name and use it the way I need it.

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