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.
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. 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.