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

Als koploper wil ik aanvullingen op Partij die nog niet in het informatiemodel van VNG zitten #69

Closed
20 tasks done
joeribekker opened this issue Oct 9, 2023 · 1 comment · Fixed by #139
Closed
20 tasks done

Comments

@joeribekker
Copy link
Member

joeribekker commented Oct 9, 2023

.. zodat ik mijn praktische use case kan invullen.

Opgevoerd bij VNG: VNG-Realisatie/klantinteracties#272

Tasks

  • Add model Categorie with fields:
  • naam (charfield, max_length=80)
  • beginDatum (datefield, default=creation date if empty/not provided in API-call)
  • eindDatum (datefield, default=None)
  • Add FK from Categorie to Partij (null=True)
  • Add model to admin (as inline of Partij)
  • Add as read-only inline array to Partij
  • Make sure the resource description says: "Let op: Dit endpoint is EXPERIMENTEEL." en hetzelfde geldt bij de description van de read-only inline bij Partij: "Let op: Dit attribuut is EXPERIMENTEEL."

Update op taken:

In retrospect, you can link a category with start/end date per party, so we need to refactor a bit.

  • Add new model, CategorieRelatie met API resource, marked with "EXPERIMENTEEL" like above.
  • Move the fields to this new model:
    • beginDatum (datefield, default=creation date if empty/not provided in API-call)
    • eindDatum (datefield, default=None)
  • Add FK from CategorieRelatie to Categorie (null=True)
  • Add FK from CategorieRelatie to Partij to Partij (null=True)
  • Remove the FK from Category to Partij

Achtergrond

Uit Excel (1):
image

Uit Excel (2):
image

Dit stond verkeerd in de Excel (Eric) en moet niet aan Externe Identificatie maar aan Partij gekoppeld zijn.
image

  • [ ]

Discuss

  • Categorie of CategorieRelatie? -> IM model benaming. Is gewoon categorie.
  • Wat is CategorieType? Is dat een enumeratie of vrije tekst? -> Door gemeente te onderhouden lijst
  • Moet Categorie niet naam heten? Het heeft al een relatie met "Categorie" -> check
  • Wat doen we met Geldigheid ivm historische gegevens die we niet opslaan? -> een categorie heeft een levensduur en staat eigenlijk los van overige historische gegevens.
  • Wat is Looptijdtype? -> staat hieronder uitgelegd.
@EvdMerwe
Copy link

De Excel is hier misschien niet duidelijk genoeg. Moeten we volgende week even doornemen. Ook is de naamgeving niet helemaal correct meer.

Hieronder de definitie van Looptijdtype zoals wij het in ons model gebruiken:

Looptijdtype definitie

@joeribekker joeribekker added this to the Release 0.7.0 milestone Nov 6, 2023
@joeribekker joeribekker modified the milestones: Release 0.7.0, Release 0.6.0 Dec 21, 2023
@bart-maykin bart-maykin moved this from Todo to In progress in Open Klant development Jan 11, 2024
@bart-maykin bart-maykin moved this from In progress to Implemented in Open Klant development Jan 12, 2024
bart-maykin added a commit that referenced this issue Jan 26, 2024
✨ [#69] added categorie endpoint and atribute on partij
@github-project-automation github-project-automation bot moved this from Implemented to Done in Open Klant development Jan 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants