Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Sep 03 20:23
    Travis simplyGits/MagisterJS (master) fixed (377)
  • Sep 03 20:21
    Travis simplyGits/MagisterJS (master) still failing (377)
  • Sep 03 20:15
    Travis simplyGits/MagisterJS (master) still failing (377)
  • Sep 03 20:14
    lieuwex closed #149
  • Sep 03 20:14
    lieuwex commented #149
  • Sep 03 20:14

    lieuwex on master

    Switch to OpenID library Removed unnessecary util functi… Reject with AuthError (compare)

  • Sep 03 19:08
    Sj3rd commented #149
  • Sep 03 19:06
    LevitatingBusinessMan commented #149
  • Sep 03 15:33
    idiidk review_requested #149
  • Sep 03 15:33
    idiidk review_requested #149
  • Sep 03 15:31
    idiidk commented #149
  • Sep 03 15:31
    idiidk commented #149
  • Sep 03 15:30
    idiidk synchronize #149
  • Sep 03 15:27
    idiidk synchronize #149
  • Sep 03 15:05
    Travis simplyGits/MagisterJS (master) broken (376)
  • Sep 03 15:04
    lieuwex closed #147
  • Sep 03 15:04
    lieuwex commented #147
  • Sep 03 15:02

    lieuwex on master

    Fixes something being undefined… (compare)

  • Sep 03 15:02
    lieuwex closed #148
  • Sep 03 14:50
    idiidk review_request_removed #149
Lieuwe Rooijakkers
@lieuwex
ah dan is dat veranderd ofzo waarschijnlijk
je kan er een issue op github voor maken en dan zou iemand het kunnnen fixen
Sj3rd
@Sj3rd
ig
Sj3rd
@Sj3rd
ahh gevonden
ik zal even een issue maken
Lieuwe Rooijakkers
@lieuwex
nieuwe versie staat op npm
meteen even dependencies geupdate
Sj3rd
@Sj3rd
Magister weer geddossed?
Jules
@julespalles
image.png
Jup
Ik vind het nu gewoon zielig worden
Sj3rd
@Sj3rd
Yep...
Sj3rd
@Sj3rd
@lieuwex hoe vraag je absenties op? Kan t nergens vinden in de docs...
Lieuwe Rooijakkers
@lieuwex
@Sj3rd niet: simplyGits/MagisterJS#104 :)
als iemand zich verveeld kan je het natuurlijk implementeren ;)
v1 had het wel maar v2 nog niet
Sj3rd
@Sj3rd
Ok ik heb een PR gemaakt met de absences erin, en een PR met de geupdate test.js
Lieuwe Rooijakkers
@lieuwex
oh ik zag alleen maar de tests PR en ik dacht dat ik even mentaal aan het afbreken was
maar vandaar :P
kaas1551
@kaas1551
Is deze lib helemaal opnieuw geschreven? Oude projectjes werken niet meer, en de "quickstart" code werkt trouwens ook niet (getSchools is not a function)
Sj3rd
@Sj3rd
Import je getSchools wel?
kaas1551
@kaas1551
Ja.
Lieuwe Rooijakkers
@lieuwex
binnenkort komt @tomsmeding met een verrassing 🤔

@kaas1551

heb je de hele code die je probeert te draaien? En wat zegt npm ls --depth 0 in de root van je project?

Sj3rd
@Sj3rd
Oehh spannend, een verrassing 🙃
Lieuwe Rooijakkers
@lieuwex

de verrassing is hier! @magisterjs/dynamic-authcode heeft nu een super coole fetcher en parser geschreven door @tomsmeding. Het haalt automatisch de JS code op van magister en doet dan een beetje AST magie en vormt daarmee de authcode. Als fallback hebben we dan natuurlijk nog de oude manier.

Deze manier van fetchen was een van de eerste voorgestelde manieren, maar ik was onzeker of de codestructuur hetzelfde bleef, dat lijkt het nu te doen. En omdat deze techniek daadwerkelijk de code parsed moet het ook wel goed blijven werken

Tom Smeding
@tomsmeding
dus voor als je je afvraagt waarom dynamic-authcode nu een dependency heeft op @babel/parser: het doet coole shit met magister's js code :)
Sj3rd
@Sj3rd
Dus je hoeft niet de hele tijd meer https://www.npmjs.com/package/@magisterjs/dynamic-authcode te updaten?
Tom Smeding
@tomsmeding
Als je https://www.npmjs.com/package/@magisterjs/authcode bedoelt: klopt, maar dat gaat voor t moment nog wel gebeuren, voor het geval de nieuwe code faalt bij een magister update of iets dergelijks
Sj3rd
@Sj3rd
Nja, ik gebruik nu gewoon t headless browser scriptje, duurt niet eens heel lang :)
Tom Smeding
@tomsmeding
Ik verwacht dat mijn code niet zo snel zal breken, maar het kan natuurlijk altijd
Een hele browser draaien op de server wil niet iedereen ;)
Sj3rd
@Sj3rd
En kan soms niet eens haha
Vincent Brouwer
@idiidk
@tomsmeding Dit is echt heel vet man, props! Dit is nou echt zon creatieve oplossing waar ik op de een of andere manier heel vrolijk van word
Tom Smeding
@tomsmeding
:D
Jules
@julespalles
Nice man @tomsmeding! Puur als uitdaging gemaakt of maak je zelf ook gebruik van MagisterJS?
Tom Smeding
@tomsmeding
Ik ben ondertussen masterstudent, dus nee ik maak er zelf geen gebruik meer van
De aanleiding was meer: Lieuwe vertelde me over het onzinverhaal
~ met de authcode
(waarom verstuurt gitter het bericht als je boven het tekstvak tikt?)
Naja dat dus, en ik dacht dit is absurd, kun je niet gewoon de code reverse-engineeren en zien hoe ze die code genereren
Ziend dat dat niet werkte maar dat de generatiecode wel echt super simpel is, dacht ik "dit moet toch kunnen" :p — en voilà
Sj3rd
@Sj3rd
"Super simpel" -> schrijft lib van 3000 regels😂
Tom Smeding
@tomsmeding
790 ;)
Andrew
@Editandrew_twitter
hey! werkt dit ook met Angular?
Sj3rd
@Sj3rd
Ligt magister er bij jullie ook uit? Ook in de browser krijgt de officiele client een error bij de password request
Sj3rd
@Sj3rd

Accreditation

The vendor of Magister, Iddink Digital, requires that all users of the API are accredited parties: this means that the company wants to check that you conform to their guidelines regarding security and personal data management. After registration with Iddink Digital, you will receive an authentication code (authcode) which is required by MagisterJS. For registration, send an email to: partners@iddinkgroup.com.
^ wat is dit ineens haha
Sj3rd
@Sj3rd
is dat iemand al gelukt?
Vincent Brouwer
@idiidk
Yoo, @lieuwex