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.

Sign up for GitHub

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

Jump to bottom

gnd_pid / pid #32

Closed
pronguen opened this issue Jun 17, 2019 · 1 comment
Closed

gnd_pid / pid #32

pronguen opened this issue Jun 17, 2019 · 1 comment
Assignees
@reropag @rerowep
Labels
bug Something isn't working f: data

Comments

@pronguen
Copy link

The JSON instance of a MEF GND person has two properties for the same identifiers:

  • gnd_pid
  • pid

gnd_pid is not in the JSON schema.

The same problem applies to BNF, RERO and VIAF persons.

The model should be simplified in order to have only one pid property used for persons from any sources.
@pronguen pronguen added the bug Something isn't working label Jun 17, 2019
@iGormilhit iGormilhit mentioned this issue Jun 24, 2019
Closed
@iGormilhit
Copy link

5946d52 closes it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@reropag reropag

@rerowep rerowep

Labels
bug Something isn't working f: data
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@iGormilhit @pronguen @reropag @rerowep