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

[Multiple Properties] - Add support for multiple values #166

Open
DannyGJdeJong opened this issue Feb 6, 2025 · 0 comments
Open

[Multiple Properties] - Add support for multiple values #166

DannyGJdeJong opened this issue Feb 6, 2025 · 0 comments

Comments

@DannyGJdeJong
Copy link
Collaborator

Add support for multiple values in the backend for the following properties:

  • CatalogueReference
  • MadeInPresentingDate
  • MadeInCentury
  • MadeInPeriod
  • ProvenanceAcquisition
  • ProvenanceAcquisitionMethod
  • ProvenanceRemarks
  • ProvenanceText

We do this as described here: https://github.com/Q42/Rijksmuseum/pull/5354#issuecomment-2633285522

Eerder comment ergens anders. Deze staat on hold want moet in meerdere stappen gedaan worden.

Als je een veld verandert van type; in dit geval dus van een string naar een list, kan het resultaat vanuit Elastic niet goed worden geparsed en krijg je een exceptie. Dit moet dus in meerdere stappen idd:

  1. Nieuwe veld met een andere naam die een list zijn en hier naartoe mappen.
  2. Oud veld verwijderen
  3. Nieuw veld toevoegen met dezelfde naam als het oude veld en list hier ook naartoe mappen
  4. Nieuwe veld met andere naam verwijderen
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant