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

Duplicate parsing errors should be grouped based on line and colum. #174

Closed
philipsens opened this issue Sep 23, 2021 · 0 comments · Fixed by #173
Closed

Duplicate parsing errors should be grouped based on line and colum. #174

philipsens opened this issue Sep 23, 2021 · 0 comments · Fixed by #173
Assignees
Labels
feature New feature or request
Projects
Milestone

Comments

@philipsens
Copy link
Member

Is your feature request related to a problem? Please describe.
If you don't close a tag properly, saxes will throw a parse error for each character that "isn't supposed to be there".

Describe the solution you'd like
There should be one error message stating a start line and end line in which the error occurred.

Describe alternatives you've considered
@chiefruds: Alternatively for filtering the errors in the canvas you can also filter the errors here on the line and only add one unique error for each line.

Additional context
Discussion started: #173

@philipsens philipsens added the feature New feature or request label Sep 23, 2021
@philipsens philipsens added this to the Angular MVP milestone Sep 23, 2021
@philipsens philipsens added this to To do in Sprint 20 via automation Sep 23, 2021
@philipsens philipsens moved this from To do to In progress in Sprint 20 Sep 23, 2021
@philipsens philipsens self-assigned this Sep 23, 2021
@philipsens philipsens moved this from In progress to Review in progress in Sprint 20 Sep 23, 2021
Sprint 20 automation moved this from Review in progress to Done Sep 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
No open projects
Development

Successfully merging a pull request may close this issue.

1 participant