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

feat: excel extractor column selection #433

Merged
merged 3 commits into from
Mar 15, 2024

Conversation

carlbrugger
Copy link
Contributor

Please explain how to summarize this PR for the Changelog:

This PR fixes column selection when there is a blank header column by removing the column with a blank header rather than the last column.

Tell code reviewer how and what to test:

Extract an Excel file that has an empty header column such as this example:
import_empty_header.xlsx

This PR will produce:
Screenshot 2024-03-07 at 12 05 31

instead of this:
Screenshot 2024-03-07 at 12 06 09

Closes: https://github.com/FlatFilers/support-triage/issues/1094

@carlbrugger carlbrugger requested a review from bangarang March 7, 2024 18:07
@carlbrugger carlbrugger force-pushed the feat/excel-extraction-column-selection branch from 1ec5e4d to 7944e98 Compare March 15, 2024 17:17
@carlbrugger carlbrugger force-pushed the feat/excel-extraction-column-selection branch from 7944e98 to 7f3eed5 Compare March 15, 2024 17:18
@carlbrugger carlbrugger enabled auto-merge (squash) March 15, 2024 17:18
Copy link

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@flatfile/[email protected] Transitive: environment, eval, filesystem, network +35 14 MB flatfileinfra

🚮 Removed packages: npm/@flatfile/[email protected], npm/@flatfile/[email protected]

View full report↗︎

@carlbrugger carlbrugger merged commit 6cc4ac2 into main Mar 15, 2024
4 checks passed
@carlbrugger carlbrugger deleted the feat/excel-extraction-column-selection branch March 15, 2024 17:38
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