csrdelft / csrdelft.nl

De webstek van Civitas Studiosorum Reformatorum, wordt onderhouden door de PubCie.
https://csrdelft.nl
17 stars 11 forks source link

Schrijf GoogleSync om naar People api #605

Open qurben opened 4 years ago

qurben commented 4 years ago

De People api wordt wel ondersteund door google/apiclient-services. Dit maakt het allemaal een stuk makkelijker onderhoudbaar (en we moeten toch dependen op google/apiclient-services).

Zie https://developers.google.com/resources/api-libraries/documentation/people/v1/php/latest/class-Google_Service_PeopleService.html voor docs.

qurben commented 4 years ago

Heb hier net even een beetje naar gekeken. De People API werkt erg makkelijk met de huidige setup van Google_Client.

Een uitdaging is wel dat de People API niet makkelijk alle leden van een contactGroup te pakken kan krijgen, je krijgt alleen een lijst id's. En je kan maximaal 50 contacten ophalen. Hier voor moet de logica omgebouwd worden om het ophalen van contacts te batchen. We moeten hier dan wel kijken of dit niet te langzaam wordt om iedere keer als een profiel bekeken wordt te doen, maar misschien kan hier ook wat gecached worden, of de api-calls alleen uitvoeren als er om gevraagd wordt.

Misschien is het ook een idee om op de server ook dingen bij te houden, zoals welke profielen door wie ge-exporteerd zijn en welke resourceName en etag bij de ge-exporteerde contacten hoort. Maar dit maakt de boel wel een stuk ingewikkelder.

qurben commented 3 years ago

Heeft nu wat hogere prio gekregen omdat de huidige API daadwerkelijk deprecated is en binnenkort waarschijnlijk helemaal niet meer zal werken.

sentry-io[bot] commented 3 years ago

Sentry issue: STEK-75