Bug 306495 Comment 68 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Thanks Philipp for the feedback.  I've just uploaded a new revision that: 
- addresses almost all of your comments
- has a working first pass on the de-xpcom parts like we discussed
- is now working for subscribing to network calendars
- includes some other minor improvements

(In reply to Philipp Kewisch [:Fallen] [:📆][:🧩] from comment #67)
> 
> When I wrote this, I was trying to keep extensibility in mind. Add-ons may be adding different types of calendars, some can do autodetection based on few information, others might require a specific URL, others might not need anything at all. The birthdays case was for example to account for the ThunderBirthday extension.
> 
> Even if these are empty (and thus hidden) now, it would allow provider extensions to be more flexible.

Ah, okay, that makes sense.  I have some questions, so will try to set up a meeting to discuss extensibility details and a few other items like the TODOs for `filterprovider` and `preferprovider`, etc.
Thanks Philipp for the feedback.  I've just uploaded a new revision that: 
- addresses almost all of your comments
- has a working first pass on the de-xpcom parts like we discussed
- is now working for subscribing to network calendars
- includes some other minor improvements

(In reply to Philipp Kewisch [:Fallen] [:📆][:🧩] from comment #67)
> 
> When I wrote this, I was trying to keep extensibility in mind. 

Ah, okay, that makes sense.  I have some questions, so will try to set up a meeting to discuss extensibility details and a few other items like the TODOs for `filterprovider` and `preferprovider`, etc.
Thanks Philipp for the feedback.  I've just uploaded a new WIP revision that: 
- addresses almost all of your comments
- has a working first pass on the de-xpcom parts like we discussed
- is now working for subscribing to network calendars
- includes some other minor improvements

(In reply to Philipp Kewisch [:Fallen] [:📆][:🧩] from comment #67)
> 
> When I wrote this, I was trying to keep extensibility in mind. 

Ah, okay, that makes sense.  I have some questions, so will try to set up a meeting to discuss extensibility details and a few other items like the TODOs for `filterprovider` and `preferprovider`, etc.

Back to Bug 306495 Comment 68