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

Make sure we are not generating invalid creation/publication event dates #3469

Closed
lwrubel opened this issue Feb 8, 2024 · 2 comments
Closed

Comments

@lwrubel
Copy link
Contributor

lwrubel commented Feb 8, 2024

Once sul-dlss/cocina-models#681 is merged, we should be sure we are not creating invalid creation/publication dates that have a date value alongside a structuredValue range.

There were six works (e.g. https://argo.stanford.edu/view/yy114sp3047) that were caught with validation and have description.event.date such as:

         {
                "type": "creation",
                "date": [
                   {
                        "structuredValue": [
                           {
                                "value": "2024-01-03",
                                "type": "start"
                            },
                           {
                                "value": "2024-02-02",
                                "type": "end"
                            }
                        ],
                        "type": "creation",
                        "encoding": {
                            "code": "edtf"
                        }
                    }
                ]
            }

The existing H2 works are being remediated by @arcadiafalcone and @amyehodge (see sul-dlss/argo#4279).

@lwrubel
Copy link
Contributor Author

lwrubel commented Feb 8, 2024

It doesn't seem like our current code will generate the invalid value when an interval is used for creation or publication dates.

See also this example, which has a valid creation date: https://argo-qa.stanford.edu/view/yz466fs4308

@lwrubel
Copy link
Contributor Author

lwrubel commented Feb 8, 2024

When @amyehodge redeposited the problem items, the cocina was valid, so closing this issue.

@lwrubel lwrubel closed this as completed Feb 8, 2024
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