Gi 18 Polish Dataclassify object

GerwinBosch commented 7 years ago

Polished data classify component Moved dataprocessing methods to their own files and created tests for them Added ES-Lint styling

reinvantveer commented 7 years ago

"Pick a file" geeft geen informatie over wat voor soort bestand hier wordt verwacht. Alleen dat een .png niet goed is :)

reinvantveer commented 7 years ago

Heb je een bestand waarmee dit zou moeten werken? Dan kan je die mee committen voor testdoeleinden. Kan je uiteindelijk met Phantomjs, Caspar.js of Browserling, Selenium, whatever testen.

GerwinBosch commented 7 years ago

@reinvantveer

Het doel van deze PR was het verbeteren van de DataClassifyView. Dit is gedaan door de vorige implementatie te vervangen door een tabel met één checkbox en wanneer je die aanklikt je een popup krijgt waarbij je eerst een base-uri(optioneel) kan ingeven en vervengens op volgende kan klikken. Vervolgens moet de gebruiker een classificatie aangeven.
Daarnaast is tijdens de ontwikkeling ES lint toegevoegd waarbij iedere file is aangepast
Test suite is uitgebreid en zou nu niet moeten falen
Twee test files toegevoegd in de test map misschien in een later stadium de tests ook naar deze map verplaatsen
De data input text valt buiten de scope van deze pull request

PDOK / LinkDaLe

Gi 18 Polish Dataclassify object #3