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

'Download alle algoritmes (.csv)' werkt niet #28

Open
jraat001 opened this issue Nov 20, 2023 · 2 comments
Open

'Download alle algoritmes (.csv)' werkt niet #28

jraat001 opened this issue Nov 20, 2023 · 2 comments

Comments

@jraat001
Copy link

Sinds gisteren doet de download link voor alle algoritmes als CSV het niet meer. Als je deze link opent (https://algoritmes.overheid.nl/api/file/algoritme) krijg je '{"detail":"Not Found"}'. Ik heb geen idee waar het aan ligt, maar hopelijk kan iemand hier er iets mee. Als ik dit beter ergens anders kan melden hoor ik het ook graag.

Hoe te reproduceren?

  1. Ga naar het algoritmeregister
  2. Klik op Algoritmes in de navigatie-bar
  3. Druk op de knop 'Download alle algoritmes (.csv)'

Wat gebeurt er nu?

Je krijgt een foutmelding ('{"detail":"Not Found"}').

Wat zou er moeten gebeuren?

Er zou een CSV gedownload moeten worden met alle algoritmes uit het register.

@floort
Copy link

floort commented Nov 21, 2023

Ik liep net tegen hetzelfde probleem aan maar bij het zoeken naar een mogelijke oorzaak kwam ik erachter dat de code die live draait niet dezelfde lijkt te zijn als die hier beschikbaar is. Misschien nog een kleine reminder voor het ontwikkelteam om de broncode van het register te delen in plaats van een oude versie van de broncode én ook individuele commits waarin afgelezen kan worden wat er veranderd is zodat het zoeken naar de oorzaak van een probleem makkelijker is.

Het lijkt erop https://algoritmes.overheid.nl/api/file/algoritme/NLD wel werkt of https://algoritmes.overheid.nl/api/file/algoritme/ENG voor als je een lege CSV wilt. Meer kan ik niet vinden.

Latere toevoeging: Ik heb code die van dit API endpoint afhankelijk is. Niet dat daar nu zo'n kritiek proces van afhankelijk is, maar misschien is het een goed idee om aanpassingen die de API breken expliciet aan te kondigen. Het lijkt erop dat de front-end code ook niet op deze aanpassing is voorbereid en dan kan je dat van externen al helemaal niet verwachten.

floort pushed a commit to floort/Algoritmeregister that referenced this issue Dec 8, 2023
Resolve "Zoekresultaten in KOOP stijl"

Closes MinBZK#28

See merge request ictu/devops/algoritmeregister/algoritmeregister!10
floort pushed a commit to floort/Algoritmeregister that referenced this issue Dec 8, 2023
Resolve "Dashboard pagina, design van Roos"

Closes MinBZK#31, MinBZK#7, #44, MinBZK#36, MinBZK#39, MinBZK#22, MinBZK#29, MinBZK#14, MinBZK#27, MinBZK#28, MinBZK#1, MinBZK#8, MinBZK#5, and MinBZK#3

See merge request ictu/devops/algoritmeregister/algoritmeregister!27
@floort
Copy link

floort commented Jan 8, 2024

Er lijkt nu een CVS download optie in de user interface te zijn die waarschijnlijk na de laatste commit in deze repository hersteld of toegevoegd. Misschien verstandig om het issue nog even open te laten staan tot er een fatsoenlijke commit is om naar te verwijzen.

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

No branches or pull requests

2 participants