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

[BUG] EXTRACTION FORMAT / DOCUMENT FORMAT MISMATCH: As Tim, I want to be able to safely change the format of an extraction, without it crashing the transformation preview, so that I don't get stuck trying to develop my pipeline. #59

Merged
merged 9 commits into from
Feb 19, 2024

Conversation

richardmatthewsdev
Copy link
Contributor

Acceptance Criteria

  • Extraction formats can be safely changed without causing the Transformation preview to crash.

Context

If you create an extraction that is XML and extract some documents. If you then change the format of the extraction to be JSON it will crash the pipeline page as it raises an error when it attempts to parse the document from the file system with the wrong language parser.

Fix issue with initial display of pipeline transformation view when the format of the extraction does not match the format of the extracted documents.
Prevent extraction preview modal from crashing when there is a mismatch between the extraction format and the format of the document.
…from breaking when its extraction job is deleted
Copy link

github-actions bot commented Feb 11, 2024

Code quality score

The code quality has not changed for this PR.

Ruby file count Similarity score ABC complexity Code smells TOTALS
base 86 6.5 5.36 16.8 28.66
this branch 86 6.5 5.36 16.8 28.66
difference 0 0.0 0.0 0.0 0.0

@paul-mesnilgrente paul-mesnilgrente merged commit e3bf47d into main Feb 19, 2024
8 checks passed
@paul-mesnilgrente paul-mesnilgrente deleted the rm/format-mismatch branch February 19, 2024 20:42
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