Closed jacquarg closed 9 years ago
@jacquarg @aenario What is the status of this PR?
it probably works, but broke tests because we logically change the content of the vcard (adding categories). Logically, client should grab the new version and all will be well. Yet, it needs to be tested functionally and if it works, tests should be adapted for it.
@jsilvestre @poupotte @clochix is there one of you who can handle it?
@jacquarg Except the fact tests are failing, do you consider it as good to merge?
@frankrousseau : it's been a while, but yes, just a build and it will be good to merge. I am disappointed by the problems with the tests as I remember that I spend some time to enrich them and make them pass. But travis as secrects...
Ok I will handle it (in fact I hope I will have enough time for it). I will include it in the work on import/export/sync of contacts.
In-progress ! depends on PR cozy-vcard#8 and next version of cozy-vcard.