Skip to content

Add person properties from agrelon.rdf (#476) - #477

Merged
fsteeg merged 2 commits into
masterfrom
476-propertySuggest
Jun 9, 2026
Merged

Add person properties from agrelon.rdf (#476)#477
fsteeg merged 2 commits into
masterfrom
476-propertySuggest

Conversation

@fsteeg

@fsteeg fsteeg commented Jun 2, 2026

Copy link
Copy Markdown
Member

Will resolve #476.

(Also contains unrelated fix for VS Code setup after Java 11 switch.)

@fsteeg
fsteeg marked this pull request as ready for review June 2, 2026 14:04
@fsteeg
fsteeg requested a review from dr0i June 2, 2026 14:05
@acka47

acka47 commented Jun 3, 2026

Copy link
Copy Markdown
Contributor

Neulich habe ich diesen Toot gelesen, und er hat mich dran erinnert, dass es ja noch viel mehr Agrelon Properties gibt, die wir nicht abdecken, oder? Wäre es nicht gut, die prophylaktisch auch schonmal zu ergänzen, wenn wir eh dran sind?

@fsteeg

fsteeg commented Jun 3, 2026

Copy link
Copy Markdown
Member Author

[...] Wäre es nicht gut, die prophylaktisch auch schonmal zu ergänzen, wenn wir eh dran sind?

Sollten alle drin sein, die agrelon.rdf wird ja komplett eingelesen, s. z.B.:

https://test.lobid.org/gnd/reconcile/suggest/property?prefix=hasLover

(Functional review eigentlich in #476.)

@dr0i dr0i assigned fsteeg and unassigned dr0i Jun 9, 2026
@fsteeg
fsteeg merged commit ff23278 into master Jun 9, 2026
1 check passed
@fsteeg
fsteeg deleted the 476-propertySuggest branch June 9, 2026 12:15
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.

Missing properties suggest results

3 participants