I've been meaning to finalize this patch, the past few times I've looked I'm not actually sure what was missing. Before I keep moving this up I am going to dump it here. Maybe it makes sense to have someone with a fresh pair of eyes look at it. Magnus, can you find someone to work on this? If you want to rebase this locally instead of applying it you can do: hg up -r c7cda013b0b433778d7f9a64b12823ab6c40f01c hg import http://... hg rebase -d comm Comments: * The tests make sense to me, but Geoff had once noted the gdata tests are hard to read so maybe these tests need more documentation or changes. * I'm not sure where it went, but I used to have the ics provider hooked up to the request classes as well. I can do some digging, but I think it shouldn't be too much work to adapt.
Bug 1546606 Comment 0 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
I've been meaning to finalize this patch, the past few times I've looked I'm not actually sure what was missing. Before I keep moving this up I am going to dump it here. Maybe it makes sense to have someone with a fresh pair of eyes look at it. Magnus, can you find someone to work on this? If you want to rebase this locally instead of applying it you can do: ``` hg up -r c7cda013b0b433778d7f9a64b12823ab6c40f01c hg import http://... hg rebase -d comm ``` Comments: * The tests make sense to me, but Geoff had once noted the gdata tests are hard to read so maybe these tests need more documentation or changes. * I'm not sure where it went, but I used to have the ics provider hooked up to the request classes as well. I can do some digging, but I think it shouldn't be too much work to adapt.