search

deltaproject / Delta

📚 Een moderne versie van Magister, gemaakt voor leerlingen.
https://delta.keesvv.nl
Mozilla Public License 2.0
13 stars 4 forks source link

Standard lint #12

Closed jvdoorn closed 5 years ago

jvdoorn commented 5 years ago

In order to keep style consistent across all JS files, I added the standard linter. Code style can be fixed by running npm test. Errors that can't be fixed automatically will have to be fixed manually.

I noticed that the linter comes across a lot of undefined errors, however, those are false-positives. In order to hide those errors, I annotated the code. In the future, we should probably look at how to keep the code clear so that we can easily see where a variable is used and defined.

There are two errors remaining, I'd fix them myself, however currently, my grades are locked (can't request them) so I would not be able to check if my fix works. Just run npm test and you'll see which ones I mean.

keesvv commented 5 years ago

Wow dit is echt heel indrukwekkend! Je verricht echt veel werk voor Delta, keep up the good work! Dit was al iets waar ik al een tijdje tegenaan zat te hikken omdat het voor mijn gevoel onmogelijk zou worden om een "nette" style aan te houden, en ik kon de code niet meer managen. Nu volgt Delta eindelijk een style guide en daar ben ik heel blij mee. :smile: :+1:

keesvv commented 5 years ago

Overigens, werken de andere features (behalve Cijfers) wel gewoon? Ik kreeg al van iemand op een andere school te horen dat de Cijfers API daar ook niet werkte, dus daar ga ik even naar kijken hoe dat zit. Als het inloggen bij jou weer werkt, ga ik #7 sluiten.

jvdoorn commented 5 years ago

Voor zover ik kan zien werken alle andere features. Mijn school vindt het alleen nodig om rond SE-weken de cijferlijst van Magister onbeschikbaar te maken (geen idee waarom, vreselijk irritant), dus wanneer je die probeert op te vragen krijg je te horen dat "er geen cijfers beschikbaar zijn". Het ligt dus niet aan Delta dat ik m'n cijfers niet kan zien.

Ik weet wel dat er problemen waren met cijfers opvragen uit voorgaande jaren. Ik weet niet of dat inmiddels is gefixed in magister.js, de issue kan je hier vinden.

Inloggen werkt alleen heb ik nog steeds geen drop-down. Ik heb geprobeerd een oplossing te vinden maar heb eerlijk gezegd geen idee wat het probleem is. Misschien dat ik er later nog een keer naar ga kijken. Dus #7 kan worden gesloten wat mij betreft, #6 zal voorlopig nog openblijven vrees ik.

keesvv commented 5 years ago

Ik heb #7 gesloten. Over de drop-down gesproken, ik ga kijken of ik een alternatief kan verzinnen in plaats van een datalist want ik vind het vervelend dat het op ieder platform anders functioneert. Over de cijfers van voorgaande jaren gesproken, ik wilde laatst een feature toevoegen waarmee je oudere cijfers kunt ophalen alleen ik kreeg simpelweg een null gereturned. Er is nog een probleem, daar zal ik nog eens een issue over openen, als je te vaak refresht loopt de DevTools console vol met errors, waarschijnlijk omdat de Magister servers het zien als een DDoS-aanval.

jvdoorn commented 5 years ago

Zodra m'n cijfers weer beschikbaar zijn kan ik kijken of het ophalen van cijfers uit voorgaande jaren weer werkt.

keesvv commented 5 years ago

Dat zal heel fijn zijn. In de tussentijd zal ik ook kijken of het werkt. Inderdaad heel irritant dat jouw cijferlijst wordt afgeschermd tijdens de SE-weken. Ik zit pas in 4V dus ik heb ze nog lang niet :smile: