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

Already on GitHub? Sign in to your account

Caldav Support WIP #84

Merged
merged 33 commits into from Apr 11, 2014

Conversation

Projects
None yet
3 participants
Contributor

aenario commented Aug 13, 2013

Hi Mike,

We needed calDAV support in node.js for cozy.io, so I went ahead and got started on it.

It still needs a lot of love, what I see missing :

  • Tests, tests, tests
  • Freebusy and Calendar queries
  • A lot of "CardDAV", "card", that I miss in comments

However, we probably won't work on it for a little while as It seems to work for our basic usage.

So this is not really a pull request per-se, but a way to fire up the discussion and to get you or may be someone else involved in this branch.

Thanks,
Romain

Owner

mikedeboer commented Oct 2, 2013

Awesome! (sorry for the late reaction :P) I'll check it out as soon as I can. But looks like quite a bit of work ;)

Owner

mikedeboer commented Nov 4, 2013

<3 this is so awesome.

I'm so sorry for not commenting earlier! I was so caught up in work for Mozilla, that I couldn't give the respects you deserve...

A question: how closely does this follow the SabreDAV implementation? Did you start there to implement this?

Contributor

aenario commented Nov 4, 2013

Hi Mike,

Yes, I ported it from SabreDAV, class and function names should be the same, although I did some heavy copy/pasting from jsDAV/CardDAV for files structure, require and event/callback handling.

Olegas and others added some commits Nov 21, 2013

@Olegas Olegas DAVACL plugin can cause deadlock in REPORT request processing cause' …
…it won't call next event handler if the specified report can't be handled by plugin.
0c4c85e
@Olegas Olegas Ability to specify own set of supported calendar component set. 3ef16db
@aenario aenario Merge pull request #2 from tensor-dev/feature/supported-calendar-comp…
…onent-set

Ability to specify own set of supported calendar component set.
729e5b4
@aenario aenario Merge pull request #1 from tensor-dev/bugfix/davacl-plugin-report-lock
DAVACL plugin can cause deadlock in REPORT request processing
ee28198
@aenario aenario fix #86 daa3189
Contributor

Olegas commented Dec 16, 2013

Owner

mikedeboer commented Apr 11, 2014

I'm merging this in, because I so desperately want this. It's a great base to work from and I welcome anyone with contributions!!!

@mikedeboer mikedeboer added a commit that referenced this pull request Apr 11, 2014

@mikedeboer mikedeboer Merge pull request #84 from aenario/caldav
Caldav Support WIP
a49e7c0

@mikedeboer mikedeboer merged commit a49e7c0 into mikedeboer:master Apr 11, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment