Lesterhuis-Training-en-Consultancy / moodle-block-user_favorites

Moodle plugin - marking pages as favorites
GNU General Public License v3.0
1 stars 4 forks source link

Moodle 4.6 | Core changes to External API #12

Closed gemguardian closed 1 year ago

gemguardian commented 1 year ago

@luukverhoeven je mag deze wat mij betreft inplannen voor Q3 - dit heeft geen haast mijn inziens omdat 4.1 nog prima werkt

Move lib/externallib.php to namespaced classes and fix coding style, etc.) https://tracker.moodle.org/browse/MDL-76583

moodle-block-user_favorites: /externallib.php

Mijn indicatie is niet meer dan 1h per plugin. Plus het uitgevoerde onderzoek van 1h.

luukverhoeven commented 1 year ago

Ingepland voor 31 maart.

gemguardian commented 1 year ago
Comment from Freshservice:
Freshservice: 154 linked successfully.
luukverhoeven commented 1 year ago

@gemguardian Aanpassing hiervan staat klaar in: https://github.com/Lesterhuis-Training-en-Consultancy/moodle-block-user_favorites/tree/8677fk16x_externalapichanges

Zal ik deze opnieuw samenvoegen met de master in een acceptatie branch?

gemguardian commented 1 year ago

@luukverhoeven is akkoord - het is overigens wel apart - dit is de versie zonder de 'verschuif' opties end aar heb ik dus géé'n issues. Ik neem overigens aan dat je zo merged dat de nieuwe functie er wel in zit? https://github.com/Lesterhuis-Training-en-Consultancy/moodle-block-user_favorites/pull/13

luukverhoeven commented 1 year ago

Doorgezet naar master branch, als het goed is staat ook de verandering van #13 nu er correct erin. Als dit goed werkt, zal ik het dan doorzetten naar Moodle?

gemguardian commented 1 year ago

@luukverhoeven graag bedankt

luukverhoeven commented 1 year ago

Doorgezet naar Moodle.org