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

rfc: weblectures-migration #1997

Merged

Conversation

zubeydecivelek
Copy link
Contributor

@zubeydecivelek zubeydecivelek commented Jan 22, 2025

closes #1981
closes #1955

@zubeydecivelek zubeydecivelek force-pushed the rfc-weblectures-migration branch from c6e392a to 4151a9a Compare January 22, 2025 13:55
@zubeydecivelek zubeydecivelek force-pushed the rfc-weblectures-migration branch from 4151a9a to 14f6aa7 Compare January 22, 2025 14:25
MANIFEST.in Outdated
@@ -75,5 +75,7 @@ recursive-include tests *.json
recursive-include tests *.mov
recursive-include tests *.mp4
recursive-include tests *.py
recursive-include rfcs *.md
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sorry we should ignore the rfcs folder here. That is why this file defines what files are included when we build/publish the package.

@zubeydecivelek zubeydecivelek force-pushed the rfc-weblectures-migration branch from 14f6aa7 to 68d38f2 Compare January 22, 2025 15:07
| Published in | 773__ | Not exist | Not exist | New fields for conference information. |
| Number of pages | 300__a | Not exist | Not exist | Will be checked with the webcast team. |
| Collaboration | 710__g | Not exist | Not exist | Add a new role to Contributors. |
| Related document | 7870_ | Not exist | Not exist | Will be `related works` as it’s done in CDS-RDM. |
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just a note: in the end in the data model this is called related_identifiers. Related works was the label chosen to display it in the UI

@zzacharo zzacharo merged commit 433c81d into CERNDocumentServer:main Jan 24, 2025
1 check passed
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.

CDS Videos Weblecture RFC Analyse migration (fields) from CDS Marc21 to CDS Videos data model
2 participants