forked from slifty/openhmis-intake
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Simplify the controller's split of file contents away from header and footer. I think that separating on `\r\n` will generally work for files we create, but I worry somewhat that other files will have `\r\n` on every line of data. Add a div to hold notifications of duplicates from the import (this will look nicer in the next commit. Consider it something of a placeholder for now). Track duplicates in a variable instead of just outputting them to the console. Insert the data from each (non-duplicate) file line into a client object. I'm a little worried that explicitly naming the column index for each element is not a portable solution, *but* HMIS explicitly details the correct format for a compliant CSV file, so maybe it's okay. What do you think, @kfogel? (I'm looking at lines 1019-1041 of index.js, here). POST the data! It saves correctly in my DB.
- Loading branch information
1 parent
77b63ac
commit a7df443
Showing
3 changed files
with
41 additions
and
11 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
a7df443
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.