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

Revisit logging of date parsing errors #188

Open
ghukill opened this issue May 31, 2024 · 0 comments
Open

Revisit logging of date parsing errors #188

ghukill opened this issue May 31, 2024 · 0 comments

Comments

@ghukill
Copy link
Contributor

ghukill commented May 31, 2024

Two immediate things come to mind when it comes to parsing / validating dates with validate_date, validate_date_range, etc.:

  1. we are forced to pull the source_record_id to pass it to the function, such that it can maybe be used for logging
  2. how helpful are the log messages themselves?

It has been discussed here and there, but probably worth revisiting these helper functions and think about how we can more seamlessly use them in code.

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