-
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
[BUG] Component
header and Freetext Value cause exception
#19
Comments
Ahh so the Invenio package needs to be referenced? |
Yes, sorry, that's missing in the steps to reproduce. Maybe just for the sake of completeness. We had an ARC that we wanted to publish. The Invenio validation package had to be added for this. Then this metadata error occurred in the pipeline. The ARC could not be published because the ARChigator page always showed the error "Could not load metadata". I can also invite you to some test ARCs if you want to. |
Okay I see. I could reproduce the error now after referencing the Invenio Test package and looking into it 👍 |
But another requirement for this to fail is, that the value in the cell needs to be freetext. Terms work just fine. |
Component
header and (seemingly) any value in cell cause exceptionComponent
header and Freetext Value cause exception
Okay this is already fixed in ARCtrl, should suffice to update the dependency in arc-to-invenio |
Okay so the Issue should fixed. @SabrinaZander @Hannah-Doerpholz would be great if you could confirm |
Maybe you could invite me to the ARC @SabrinaZander, so I can have a look? |
Since I have the same issue in my test ARC I invited you to that one if that helps. There is no "secret" data in that one @HLWeil |
Should be fixed with 0f4fba8 @Hannah-Doerpholz, I restarted your pipeline and it ran through. @SabrinaZander, could you also check? |
It ran through for my actual "data" ARC as well. Thank you! |
Thank you very much, in my test ARC the pipeline runs successfully. If you would like to investigate this further, I would be happy to invite you. It is no longer relevant for me, as we have already remade the ARC and published it. |
Yeah, I'll take a look! |
Okay, this problem is actually a mismatch between the strictness of the validation and the conversion, regarding the information in the contacts:
In this case, one person was missing the So, in principle the goal will be to align both tools to follow the same rules. I don't want to decide on which of the two approaches to take by myself, so I'll move this to a discussion. *Required fields
|
Describe the bug
ValidateAgainstHeader causes an error in everyday usecase.
To Reproduce
Steps to reproduce the behavior:
Component
building blockComponent
building blockCharacteristics
Expected behavior
Everyting's fine.
OS and framework information (please complete the following information):
Additional context
Full history of the bug occurence.
The text was updated successfully, but these errors were encountered: