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

Excel/html files should be processed directly in tabular module #228

Open
blcham opened this issue Nov 10, 2023 · 0 comments
Open

Excel/html files should be processed directly in tabular module #228

blcham opened this issue Nov 10, 2023 · 0 comments
Assignees

Comments

@blcham
Copy link
Contributor

blcham commented Nov 10, 2023

In Tabular module, we transfrom HTML and Excel files into CSV and then process them using supercsv library. We need to do that directly and not transform it. In order to do that, we need to refactor the code first.

There should be a unified interface to process tabular data using (I guess) the adapter pattern. Try to do little commits to refactor it incrementally.

A/C:

blcham added a commit that referenced this issue Nov 10, 2023
Not complete -- tests are failing here.
blcham added a commit that referenced this issue Aug 14, 2024
Not complete -- tests are failing here.
blcham added a commit that referenced this issue Aug 14, 2024
Not complete -- tests are failing here.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

2 participants