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

Added includeEmptyCells to Spreadsheet node #1249

Merged
merged 4 commits into from
Dec 12, 2020
Merged

Conversation

AxelRueweler
Copy link
Contributor

For a project I missed an option to always have all the columns represented in the JSON (for Ifs and switches and so forth).

@janober janober merged commit 35f6fe8 into n8n-io:master Dec 12, 2020
@janober
Copy link
Member

janober commented Dec 12, 2020

Great can see how that can be helpful. Thanks a lot! Got merged and released with the next version.

@RicardoE105
Copy link
Contributor

@AxelRueweler good addition. A community member was having issues with this and thought it was a bug. I added the same change and when was about to send the PR realized just already did. Thanks.

@bberenberg
Copy link
Contributor

@janober do you have an eta on the next release when this will be included?

@janober
Copy link
Member

janober commented Dec 15, 2020

Tomorrow or Thursday.

@janober
Copy link
Member

janober commented Dec 16, 2020

Got released with n8n@0.98.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.

4 participants