Skip to content

Prepro error functions#6800

Closed
anna-parker wants to merge 3 commits into
mainfrom
prepro_error_functions
Closed

Prepro error functions#6800
anna-parker wants to merge 3 commits into
mainfrom
prepro_error_functions

Conversation

@anna-parker

@anna-parker anna-parker commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

resolves #

Screenshot

PR Checklist

  • All necessary documentation has been adapted.
  • The implemented feature is covered by appropriate, automated tests.
  • Any manual testing that has been done is documented (i.e. what exactly was tested?)

🚀 Preview: Add preview label to enable

@claude claude Bot added the preprocessing Issues related to the preprocessing component label Jun 29, 2026
@claude

claude Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

This PR may be related to: #3849 (Improve/Simplify prepro error messages) and #6536 (Refactor error/warnings to be less verbose in code) — both describe simplifying/cleaning up preprocessing error messages, which matches what this PR does.

[output_field],
AnnotationSourceType.METADATA,
message=f"Internal error: network error while {action} '{subject}': {e}. Please contact the administrator.",
message=f"Internal error: network error while {action} '{subject}': {e}.",

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this changed shouldnt have happened

@anna-parker

Copy link
Copy Markdown
Contributor Author

We should really get #6537 merged and then this whole PR can be deleted and we can make a new one to specify that internal errors shoudl start and end with specific text

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

preprocessing Issues related to the preprocessing component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant