search

Z3r0byte / Magistify

All your favorite Magister tools in one app
https://magistify.nl
Apache License 2.0
2 stars 2 forks source link

Nieuwste Cijfers Bug/Feature #27

Closed ghost closed 7 years ago

ghost commented 7 years ago

In het Nieuwste Cijfers menu worden cijfers die ouder dan 6 dagen zijn, niet weergegeven. Klopt dit of is dit een bug? Als dit zo hoort, zou ik het fijn vinden als deze maximale leeftijd instelbaar zou zijn.

ghost commented 7 years ago

@Z3r0byte Dit is geen duplicate van #25, hoewel de titel dat wellicht suggereert

Z3r0byte commented 7 years ago

Dit hoort zo, de Magister API stuurt, als je de nieuwste cijfers opvraagt, cijfers op van maximaal een week oud.

ghost commented 7 years ago

Dat is erg merkwaardig aangezien andere Magister apps zoals Labda dit wel kunnen. Zijn er geen extra arguments/parameters die je kunt meesturen met je request zodat er oudere cijfers zichtbaar zijn?

Z3r0byte commented 7 years ago

Ik denk dat Labda (maar ik weet dit niet zeker) alle cijfers opvraagd en daarvan de nieuwste pakt, dit is echter minder efficient.

ghost commented 7 years ago

@Z3r0byte Iets wat je kunt doen, is de oude cijfers (ouder dan één week) eenmalig ophalen en elke keer dat er een nieuw cijfer is, voeg je die toe aan de database. Een mogelijk probleem is dan echter wel dat cijfers later wel nog aangepast kunnen worden en de database dan niet klopt. Misschien kun je de oorspronkelijke cijfers dan elk uur updaten, waardoor de snelheid behouden wordt

ghost commented 7 years ago

@Z3r0byte Ik begrijp dat het een lastige taak is, maar het lijkt me toch iets om in overweging te nemen. Zodra ik ooit Java geleerd heb (wat nog lang zal duren) zal ik proberen je te helpen (als dat mag) met al deze feature requests aangezien dit niet best een lange lijst aan het worden is.

Z3r0byte commented 7 years ago

@StanNL ik zal er eens naar kijken, het zou in principe niet heel moeilijk zijn om te doen, aangezien ik al gebruik maak van een database. En helpen mag altijd, dat is het idee achter open-source. :)

ghost commented 7 years ago

@Z3r0byte Misschien moet je de issue anders re-openen aangezien het weer relevant is

ghost commented 7 years ago

@Z3r0byte Hoewel... Eigenlijk hoeft dat niet want het is nu een duplicate van #12 geworden

ghost commented 7 years ago

En helpen mag altijd, dat is het idee achter open-source. :)

Ik vroeg me trouwens iets af. Github is open source en iedereen kan helpen, maar hoe? Ik begrijp dat je daarvoor bepaalde permissies binnen een repository nodig hebt, maar Android apps worden toch in Android Studio (of Eclipse of een andere IDE) gemaakt? Hoe kun je dan die bestanden aanpassen en hoe wordt dat dan gesynchroniseerd naar Android Studio?

Z3r0byte commented 7 years ago

Als iemand anders dan de eigenaar wijzigingen wilt aanbrengen in een repo, dan moet deze eerst een zogenaamde fork maken. Dit is eigenlijk gewoon een kopie. Als daarna de wijzigingen zijn aangebracht die diegene wilde maken kan deze een Pull Request (ook wel PR) maken. De eigenaar van de originele repo kan de wijzigingen dan afwijzen of accepteren. Integratie met Android Studio is trouwens heel makkelijk, daar zijn gewoon instellingen voor.

ghost commented 7 years ago

@Z3r0byte Oh bedankt voor de uitleg! Maar als je dus een fork maakt, moet je die fork downloaden naar Android Studio, vervolgens de wijzigingen doen, en dan alles weer van Android Studio naar Github uploaden, en dan pas de PR doen? Nou ik zorg er eerst maar voor dat ik Java ken en dan zien we wel verder. Android Studio werkt inmiddels... nu nog een http request leren sturen, maar daar is stack overflow voor

ghost commented 7 years ago

Dit is inmiddels een duplicate van #12

Z3r0byte commented 7 years ago

Je kunt als je in Android Studio een nieuw project aanmaakt, kiezen voor 'Import from Version Control' je kunt dat inloggen met je GitHub account en je krijgt dan je repo's te zien. Je kunt dan kiezen welke je kunt downloaden en Android Studio doet de rest. Als je dan wijzigingen hebt gemaakt kun je op Commit Changes klikken (volgens mij is dat de groene pijl omhoog boven in je scherm).