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

MCF Browser - TMCF Parsing Error Checking #74

Open
clincoln8 opened this issue Dec 17, 2020 · 0 comments
Open

MCF Browser - TMCF Parsing Error Checking #74

clincoln8 opened this issue Dec 17, 2020 · 0 comments

Comments

@clincoln8
Copy link
Contributor

clincoln8 commented Dec 17, 2020

The tmcf+csv parser does not currently check for error cases while parsing. Error checking should be included in this step to give the user a precise location of where/why the parsing error is occurring so they can fix their file accordingly.
The parsing errors table displayed to the user should indicate which line of the TMCF and which row in the CSV file is causing the error.

To do this, ParseTmcf class must check for returned the logged errors. back-end/server-api.js will need to be modified to capture the errors found by ParseTmcf. If capturing these errors causes a change to the structure of App's state parsingErrs, then App.jsx will need to be modified. Finally, ParsingErrsTable.jsx component will need to be modified in order to display the CSV row data.

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

No branches or pull requests

1 participant