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

ODRC: Publicatie uitbreiden met informatiecategorieën #48

Closed
3 tasks done
MarcoKlerks opened this issue Sep 23, 2024 · 4 comments · Fixed by #114 or #119
Closed
3 tasks done

ODRC: Publicatie uitbreiden met informatiecategorieën #48

MarcoKlerks opened this issue Sep 23, 2024 · 4 comments · Fixed by #114 or #119

Comments

@MarcoKlerks
Copy link
Contributor

MarcoKlerks commented Sep 23, 2024

In #35 hebben we gerealiseerd dat een publicatie gecreëerd kan worden. In #14 ontsluiten we de informatiecategorieën. In deze story zorgen we ervoor dat een publicatie aan een of meer informatiecategorieën gekoppeld kan worden.

Acceptatie criteria

  • Het datamodel wordt uitgebreid zodat op een publicatie een of meer informatiecategorieën vastgelegd kan worden.
  • De API wordt uitgebreid, zodat bij het creëren, raadplegen en muteren van een publicatie ook informatiecategorieën meegegeven kunnen worden.
  • Een publicatie moet aan minstens één informatiecategorie gekoppeld zijn.
  • De API specificatie is bijgewerkt (ReDoc, Swagger)
  • De documentatie is bijgewerkt (Read the Docs)
  • Een grafisch datamodel van de database is beschikbaar / bijgewerkt (op GitHub of Read the Docs of elders)

Taken

  • Ensure a list of UUIDs of information categories can be sent during create/update of publications
  • Ensure the information categories can be set/selected in the admin (horizontal select or raw ID widget)
  • No data migration required for existing data, we'll fix it up via the admin
@MarcoKlerks MarcoKlerks added this to the 01. ODRC: Basis milestone Sep 23, 2024
@MarcoKlerks MarcoKlerks moved this to Refinement in GPP-Woo Sep 23, 2024
@sergei-maertens sergei-maertens moved this from Refinement to Ready in GPP-Woo Oct 30, 2024
@bart-maykin bart-maykin moved this from Ready to In progress in GPP-Woo Oct 30, 2024
bart-maykin added a commit that referenced this issue Nov 1, 2024
…admin, and updated the admin tests to accomidate for it.
bart-maykin added a commit that referenced this issue Nov 1, 2024
…e api. Added tests to verify that in the admin and api informatie categorieen can't be empty
bart-maykin added a commit that referenced this issue Nov 5, 2024
… we set informatie categorieen in the publication factory, added documentation in the admin page for the newly added field.
@bart-maykin bart-maykin moved this from In progress to In review in GPP-Woo Nov 5, 2024
@github-project-automation github-project-automation bot moved this from In review to Done in GPP-Woo Nov 5, 2024
@sergei-maertens
Copy link
Contributor

Testing required

@github-project-automation github-project-automation bot moved this from Done to In progress in GPP-Woo Nov 5, 2024
@sergei-maertens sergei-maertens moved this from In progress to Testing in GPP-Woo Nov 5, 2024
@sergei-maertens
Copy link
Contributor

@MarcoKlerks het API aspect kan je niet testen, dat zal @felixcicatt of een collega moeten doen, maar je kan de admin wel testen :)

@MarcoKlerks
Copy link
Contributor Author

Testbevindingen:

  1. Ik kan in de admin-interface niet goed zien welke informatiecategorieën gekoppeld zijn aan een publicatie. Ik kan dit als beheerder dus ook niet goed corrigeren. Ik zie wel een veld Informatiecategorieën met enkele cijfers (array), maar ik kan niet achterhalen welk cijfer welke informatiecategorie is.
    Voorkeur: ik zie namen i.p.v. cijfers.
    Alternatief: ik zie het ID / cijfer in de lijst informatiecategorieën.

@github-project-automation github-project-automation bot moved this from In review to Done in GPP-Woo Nov 5, 2024
@github-project-automation github-project-automation bot moved this from Done to In progress in GPP-Woo Nov 5, 2024
@sergei-maertens sergei-maertens moved this from In progress to In review in GPP-Woo Nov 5, 2024
@sergei-maertens sergei-maertens moved this from In review to Testing in GPP-Woo Nov 5, 2024
@sergei-maertens
Copy link
Contributor

@MarcoKlerks - this can be re-tested! Note that we're aware that long labels get cut-off. There exist workarounds for this, but in my opinion those aren't worth the effort at the moment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
3 participants