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

Stabiele URL's van Dashboard-pagina's (ticket 85771, 16/12/2019) #83

Open
joostschouppe opened this issue May 25, 2020 · 1 comment
Open

Comments

@joostschouppe
Copy link
Contributor

Een Dashboard-pagina hoort bij een Thema-code met bijhorende Naam. Er is ook een “Name in Dashboard” die je zonder meer kunt wijzigen. Die naam is zichtbaar voor een gebruiker als die de pagina bezoekt. Een collega met oog voor detail had me de zeer terechte opmerking gemaakt dat de titel hier https://provincies.incijfers.be/Dashboard/Basisonderwijs---submap/ een beetje onnozel is.

Ik dat dus aangepast. MAAR het veld “Name in Dashboard” bepaalt ook de URL van de pagina. Gevolg: dode links, en een melding van een eindgebruiker (gelukkig!). Mogelijke oplossingen:

  • Pas gewoon de naam aan, en kuis daarna de links op. MAAR je kan onmogelijk weten wie er allemaal naar je linkt, dus je veroorzaakt nog altijd een hoop 404’s
  • Geef in Dashadmin een waarschuwing: “opgelet, als u deze naam aanpast, dan zullen links naar deze pagina niet meer werken”. Dan ben je op zijn minst gewaarschuwd, ook als je een slecht geheugen hebt. Maar dan kan je dus (als je een beetje klantvriendelijk wil zijn) nooit meer de naam van een pagina aanpassen.
  • Zorg ervoor dat oude URL’s doorverwijzen naar nieuwe. Wellicht problematisch, want er zouden dubbels op kunnen duiken
  • Koppel achterliggend Name in Dashboard los van de URL
@joostschouppe
Copy link
Contributor Author

joostschouppe commented Mar 8, 2021

Bijkomend issue (zie ticket 81634): statistieken zijn complexer dan ze zouden moeten zijn omdat je een dashboard (vb bevolking) kan raadplegen via https://provincies.incijfers.be/dashboard/dashboard/bevolking/ en https://provincies.incijfers.be/dashboard/bevolking/
Suggestie: zet dashboard gewoon rechtstreeks onder het hoofddomein.

@joostschouppe joostschouppe added this to the Prioriteitenlijst milestone Mar 15, 2022
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