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.

Sign up for GitHub

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

Jump to bottom

fix: relax elastic condition for partial dates #8509

Merged

makelicious merged 2 commits into develop from fix/date-indexing

Jan 28, 2025

Collaborator

makelicious commented Jan 28, 2025

Relax condition for 'date' in elasticsearch to support for partial dates saved in 'draft' mode.

Going forward we are building stricter validation for data types. We still need to think whether we should index items with bad data.


          fix: relax elastic condition for partial dates

makelicious requested a review from rikukissa

January 28, 2025 12:50

github-actions bot commented Jan 28, 2025

Oops! Looks like you forgot to update the changelog. When updating CHANGELOG.md, please consider the following:

Changelog is read by country implementors who might not always be familiar with all technical details of OpenCRVS. Keep language high-level, user friendly and avoid technical references to internals.
Answer "What's new?", "Why was the change made?" and "Why should I care?" for each change.
If it's a breaking change, include a migration guide answering "What do I need to do to upgrade?".

rikukissa approved these changes

View reviewed changes


          Merge branch 'develop' into fix/date-indexing

1ec96f1

makelicious merged commit 1658a6f into develop

40 of 41 checks passed

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

Labels

None yet