-
Notifications
You must be signed in to change notification settings - Fork 494
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
Refactor the OAI code, Step 2 #8842
Comments
|
This is a big, elephant-sized issue. We can try and estimate it, but in the spirit of our current process - should we consider splitting it into smaller/manageable chunks instead? |
sizing:
|
I will make an attempt to create the child issue for the identifiable sub-tasks involved. |
sizing:
|
Sizing:
|
Sizing:
|
2023/09/25: Added to 6.1 milestone as per conversation during prioritization meeting. |
2023/10/23: Sized at 80 due to the scope of the work anticipated. |
2024/03/12 |
This is a followup issue for #8372 (PR #8753).
Once that is merged we can proceed cleaning up and refactoring the remaining old OAI code.
Some of that code predates Dataverse and was copy-and-pasted from the implementation in DVN.
In the process it created in some extra layers of code that duplicate each other, and should probably be significantly simplified. (That is on top of a significant amount of old code that we used to maintain on the Dataverse side, but got rid of in Step. 1 when switching to the new gdcc/xoai library).
Major pieces:
Aside from that, the overall code needs to be cleaned up and modernized.
The text was updated successfully, but these errors were encountered: