[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 4 open bugs)
Details
(Keywords: meta)
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
Reporter | ||
Updated•11 years ago
|
Comment 1•5 years ago
|
||
We should make this happen for 76, and drop libical shortly after.
Comment 2•4 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•3 years 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•3 years ago
|
Comment 4•3 years ago
|
||
Is this the correct pref to enable it? calendar.icaljs
Comment 5•3 years ago
|
||
I guess I found the place where this is controlled and can make a patch
Comment 6•3 years ago
|
||
Updated•3 years ago
|
Comment 7•3 years ago
|
||
After landing on trunk, just request beta uplift.
Comment 8•3 years ago
|
||
Please help me track checkin-needed and approval-beta when ready.
Comment 9•3 years 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•3 years ago
|
Updated•3 years ago
|
Comment 10•3 years 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•3 years ago
|
Comment 11•3 years 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•3 years 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•3 years ago
|
||
Hmm, yeah. I'll let this be marked as fixed when it lands, and just not have an assignee.
Updated•3 years ago
|
Comment 14•3 years ago
|
||
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/e833a3991cb8
enable ical.js by default. r=mkmelin
Updated•3 years ago
|
Description
•