[meta] Enable ical.js by default
Categories
(Calendar :: ICAL.js Integration, task, P2)
Tracking
(thunderbird_esr91 wontfix)
Tracking | Status | |
---|---|---|
thunderbird_esr91 | --- | wontfix |
People
(Reporter: Fallen, Unassigned)
References
(Depends on 5 open bugs)
Details
(Keywords: meta)
Attachments
(1 file)
This bug should gather blocking issues that keep us from switching to ical.js by default. This mostly means issues that restrict usability, i.e performance issues or major errors. More to come
Reporter | ||
Updated•9 years ago
|
Comment 1•4 years ago
|
||
We should make this happen for 76, and drop libical shortly after.
Comment 2•2 years ago
|
||
Concerning https://mail.mozilla.org/pipermail/tb-planning/2020-October/008030.html, if the plan is to switch to https://github.com/mozilla-comm/ical.js/, then somebody shall tackle now the problems reported for ical.js, as this will avoid later problems within Thunderbird.
As far as I know libical is very much modified within the Thunderbird source code, compared to upstream and upstream it has very few reported problems. (Or rather, compared to its usage, the reported problems have the nature of feature requests or uncertainty of how to read RFCs). Switching to upstream libical could be also a means to increase maturity by reducing IT burden.
Comment 3•1 year ago
|
||
It has been suggest to enable this by default for TB Beta, can we do this immediately? If yes, who could handle it?
Updated•1 year ago
|
Comment 4•1 year ago
|
||
Is this the correct pref to enable it? calendar.icaljs
Comment 5•1 year ago
|
||
I guess I found the place where this is controlled and can make a patch
Comment 6•1 year ago
|
||
Updated•1 year ago
|
Comment 7•1 year ago
|
||
After landing on trunk, just request beta uplift.
Comment 8•1 year ago
|
||
Please help me track checkin-needed and approval-beta when ready.
Comment 9•1 year ago
•
|
||
(In reply to Kai Engert (:KaiE:) from comment #3)
It has been suggest to enable this by default for TB Beta, can we do this immediately? If yes, who could handle it?
I haven't seen any discussion or information on what has been tested. And what are the plans/time line surrounding enabling this. Can you add both of these to the bug report please?
Some points/questions:
- What do we know about the size of the daily users who use calendar, and how that testing has gone?
- Have we solicited beta users to test prior enabling it by default for all beta users? (I'm sure we have users who would be willing to test)
- Almost no icaljs bug reports have been fixed in the past year. And almost no open bug reports have been touched in the past year, including bugs which are marked as blockers to this bug report.
- Are there potential blocker issues at https://github.com/libical/libical/issues?
- Do we have calendar developer(s) who are committed to immediately fixing regressions reported by beta users?
- Why would we enable this for beta before doing Bug 1754417 - upgrade to ical.js 1.5.0?
Updated•1 year ago
|
Updated•1 year ago
|
Comment 10•1 year ago
|
||
We may have to delay getting it onto beta for a month, so we can address at least some of those bugs before that. Next merge is March 8.
Updated•1 year ago
|
Comment 11•1 year ago
|
||
(In reply to Magnus Melin [:mkmelin] from comment #10)
We may have to delay getting it onto beta for a month, so we can address at least some of those bugs before that. Next merge is March 8.
Sounds like we're ready to land into c-c. If we land it now, it won't be in beta until after that date.
Comment 12•1 year ago
|
||
Note that I was really just giving a helping hand with this tiny patch, I don't think this calendar meta bug should be assigned to me.
Comment 13•1 year ago
|
||
Hmm, yeah. I'll let this be marked as fixed when it lands, and just not have an assignee.
Updated•1 year ago
|
Comment 14•1 year ago
|
||
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/e833a3991cb8
enable ical.js by default. r=mkmelin
Updated•1 year ago
|
Description
•