search

SURFnet / sp-dashboard

SURFconext Service Provider Dashboard
Apache License 2.0
5 stars 2 forks source link

XML import problemen tonen #829

Closed phavekes closed 1 month ago

phavekes commented 1 month ago

This issue is imported from pivotal - Originaly created at Apr 17, 2018 by Bart Geesink

Als een XML import niet goed gaat, dan komt een generieke foutmelding naar voren, waarna in de logs gekeken moet worden waarom de import niet goed ging. Het is handiger om de output van de XML parser direct te tonen aan de gebruiker.

Code changes https://github.com/SURFnet/sp-dashboard/pull/114

phavekes commented 1 month ago 
@teunf and @bartgeesink, zoals beloofd, bij deze een voorstel van hoe de XML parser output getoond zou kunnen worden. Is dit in lijn met de verwachtingen?

De twee onderste blokken laten zien hoe langere foutmeldingen weergegeven zouden worden. (Michiel Kodde - Apr 23, 2018)

phavekes commented 1 month ago 
@michielkodde @bartgeesink Wat mij betreft ziet het er goed uit. Als ik het goed begrijp wordt iedere foutmelding in een aparte container gezet? Dus in bovenstaande voorbeeld zijn er 4 fouten? De scrollbar bij de langere foutmeldingen vind ik geweldig. (Teun Fransen - Apr 23, 2018)
phavekes commented 1 month ago

Dat klopt helemaal @teunf (Michiel Kodde - Apr 23, 2018)

phavekes commented 1 month ago

Dit is meer dan ik me er van voor had gesteld zelfs. Erg mooi dat de foutmeldingen uit elkaar getrokken worden, ook een stuk leesbaarder dan een logfile doorlezen! (Bart Geesink - Apr 23, 2018)