Skip to content

Conversation

@daften
Copy link
Contributor

@daften daften commented Jun 18, 2025

A fix for issue #629
For full transparency: This was generated with the help of Claude Sonnet, I'm not a python developer.

@FuzzyGrim
Copy link
Owner

FuzzyGrim commented Jun 18, 2025

Thank you!

I will keep:

except Exception as error:
    error_msg = f"Error processing entry: {row}"
    raise MediaImportUnexpectedError(error_msg) from error

Because if the user has used an unexpected CSV format, it should stop the import so we don't make unnecessary API requests for each media ID when they won't be able to be imported anyway.

@FuzzyGrim FuzzyGrim merged commit 3099b3e into FuzzyGrim:dev Jun 18, 2025
1 check passed
@daften
Copy link
Contributor Author

daften commented Jun 18, 2025

I’ll tackle that in a separate PR tomorrow normally!

@FuzzyGrim
Copy link
Owner

It's fine, I will do it. Thanks

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