search

Befox / cdav

Dolibarr module to access calendars and address book from CalDAV/CardDAV tools
GNU General Public License v3.0
47 stars 28 forks source link

FIX: use sabre/dav instance from Dolibarr #56

Closed marc-dll closed 1 month ago

marc-dll commented 1 year ago

Hi,

One way to fix issue #53 is to use the Sabre/DAV library from Dolibarr. I quickly tested the CardDAV sync and it seems to work.

@jpfox Does the module use some of the newer APIs of Sabre/DAV or is the old Dolibarr version enough ? I would strongly advise to test thouroughly that everything works before merging this PR anyway.

Greetings.

Bonjour,

Une manière de résoudre l'issue #53 est d'utiliser la bibliothèque Sabre/DAV de Dolibarr. J'ai testé rapidement la synchro CardDAV et ça semble fonctionner.

@jpfox Est-ce que le module utilise des APIs récentes de Sabre/DAV ou bien la version de Dolibarr est-elle suffisante ? Quoi qu'il en soit, je vous recommanderais fortement de tester en profondeur que tout fonctionne bien avant de merger cette PR.

jpfox commented 1 year ago

Merci @marc-dll pour la PR... nous allons tester. Cela semble une bonne idée d'utiliser la version présente dans Dolibarr. Le module n'utilise pas d'apoi particulièrement récente de Sabre/DAV.

marc-dll commented 10 months ago

Bonjour @jpfox ,

Avez-vous pu tester ?

De notre côté, on utilise cette branche depuis quelques mois sans problème particulier. Par ailleurs, la version embarquée dans Dolibarr a été mise à jour en version 18, avec la dernière version compatible avec PHP 7.1 (minimum requis dans Dolibarr v18).

Merci de votre retour.

Bien cordialement.