Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

FIX: use sabre/dav instance from Dolibarr #56

Merged
merged 2 commits into from
Aug 5, 2024

Conversation

marc-dll
Copy link
Contributor

@marc-dll marc-dll commented Mar 8, 2023

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
Copy link
Member

jpfox commented Mar 14, 2023

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
Copy link
Contributor Author

marc-dll commented Nov 8, 2023

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants