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

mod_ginger_rdf: Leave out empty translation strings in rdf output #736

Merged
merged 2 commits into from
Feb 4, 2025

Conversation

robvandenbogaard
Copy link
Contributor

Empty translations or entirely omitted properties should not end up as triples with empty string literal objects in the rdf rendering of resources. It doesn't add much value and clutters the view on the actual content. More so for many to 1 maps, where multiple source properties map to the same predicate (e.g. a project specific mapping where schema:description is sourced from both resource summary and body text - where summary is a default mapping but typically left blank in this project).

@mworrell mworrell self-requested a review February 4, 2025 08:09
@mworrell mworrell enabled auto-merge (squash) February 4, 2025 08:10
@mworrell mworrell merged commit d18a240 into master Feb 4, 2025
1 check passed
@mworrell mworrell deleted the no-empty-translations branch February 4, 2025 08:12
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.

2 participants