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

Further fixes to InvenioRDM to work properly with the tutorial (... and at all) #229

Merged
merged 5 commits into from
Nov 6, 2023

Conversation

led02
Copy link
Member

@led02 led02 commented Nov 2, 2023

Note that (for ... reasons) this also includes #228.

This is due to the hotfix/invenio-rdm branch is used for a concrete project already but. But for this to work, the change in the post-processing (which is part of the feature/227-adapt-tutorial-for-inveniordm branch) needed to be included.

Hence, I closed the #228 again.

Fixes #227

This might have been parametrized or implemented differently to distinguish between Invenio and InvenioRDM.
However, this way works without further coding headache...
@led02 led02 requested a review from schlauch November 2, 2023 15:21
@led02 led02 changed the title Add '_' to allowed key characters Further fixes to InvenioRDM to work properly with the tutorial (... and at all) Nov 3, 2023
@led02 led02 merged commit cdce6ac into develop Nov 6, 2023
4 checks passed
@led02 led02 deleted the hotfix/invenio-rdm branch August 2, 2024 06:46
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

Successfully merging this pull request may close these issues.

Adapt tutorial for InvenioRDM plugin
2 participants