-
Notifications
You must be signed in to change notification settings - Fork 0
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
Two works can't be edited #464
Comments
|
Hi @grahamjevon, This error was being caused by the fact that incoming dates of the format "DD/MM/YYYY" (which can be saved via Bulkrax), cannot be parsed as the the thing that parses the dates and puts them in the edit form only handles "MM/DD/YYYY". I have assumed that we want UK date formats and switched around the date parse to expect those. This will fix the error you reported (once deployed to production). I have tested this on staging with the record https://bl.bl-staging.notch8.cloud/concern/datasets/4506e7b1-4df4-4002-b21e-60acde4d6165 A caveat: The date parsing cannot handle both types of the dates... it is either one or the other. While UK format is probably the right way to go, if we have dates stored in the US format the will error in exactly the way you have reported above. |
Good news. The BL repo only has 23 records that have a BX ID and contain a date that is more than just a year. This is a manageable number to check manually when the date parser is changed. I'd be very surprised if any of the other tenants contain more than this, but I will check to be safe. But form that perspective, I think it will be ok to deploy this. I note that in the export, some of these dates are in format 06/10/2023 and some are in 2023-10-06 (See #484 for another ticket related to this). Should we prioritise one of these formats in the csv template when importing or can the repo be tweaked to ope with both? Currently, I aim to follow "2023-10-06", but am mindful that Excel likes to switch it (though we warn partners against using Excel). |
Hi @grahamjevon, thanks for checking through those. It'll cope with either DD/MM/YYYY or YYYY-MM-DD (just not both the slash-based formats at the same time). I guess that the duality has come from the original date input from the form that UB put in and the date handling from Bulkrax. TBH It would be easier if all dates to be stored the same way (even if we are being unfussy about how they are entered) #484 is a good example of needing to be aware of all possible date storage formats... I'll follow up on that ticket, but sounds like we are happy to deploy this change #478 ? (which is specifically the edit form handling DD/MM/YYYY dates as opposed to MM/DD/YYYY ones) |
Yes, we are happy to deploy this change. I have checked exports from all the tenants and there are minimal works added via BX with a full date that might be affected by this switch. And they probably need checking anyway. |
I've reviewed the metadata from the tenants and I'm happy to pass this along |
The following two works were uploaded via BX. But when I click on "Edit", I get an error:
I believe the cause of this error is an invalid date format in the "submitted to publisher" field.
Presumably the solution is to either:
The text was updated successfully, but these errors were encountered: