1901384 - Export calendar to ICS does not provide calendar NAME property

Status: RESOLVED FIXED

(Calendar :: Import and Export, enhancement)

Description

[james]

Steps to reproduce:

When exporting a calendar to .ics the name of the calendar is not included in the .ics file that is written.

Actual results:

The .ics file that is output does not provide the calendar name data, instead the only information is:

BEGIN:VCALENDAR
PRODID:-//Mozilla.org/NONSGML Mozilla Calendar V1.1//EN
VERSION:2.0

As a calendar name must be provided for any created calendar, it would be nice to have the NAME/X-WR-CALNAME properties included in the output file when exporting for additional metadata information.

If importing this file into another calendar platform the name will likely be used and autopopulated.

Comment 1

[Magnus Melin [:mkmelin]]

https://datatracker.ietf.org/doc/html/draft-daboo-icalendar-extensions-06#section-5.1
(See also https://stackoverflow.com/questions/17152251/specifying-name-description-and-refresh-interval-in-ical-ics-format)

Comment 2

[Magnus Melin [:mkmelin]]

Attachment: Bug 1901384 - Add NAME to exported ics calendar. r=#thunderbird-reviewers

Comment 3

[Martin Schröder [:mschroeder]]

This seems to be a duplicate of bug 1802808 or the other way around. There exist also other issues regarding implementation of RFC 7986 (https://datatracker.ietf.org/doc/html/rfc7986), which is the final version of the draft mentioned in comment#1.

Comment 4

[Magnus Melin [:mkmelin]]

Thanks Martin, that is very related - but a bit wider in scope.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/9c71b9dde3b2
Add NAME to exported ics calendar. r=aleca,babolivier

Comment 6

[Magnus Melin [:mkmelin]]

Comment on attachment 9408503 [details]
Bug 1901384 - Add NAME to exported ics calendar. r=#thunderbird-reviewers

[Approval Request Comment]
User impact if declined: NAME won't be used
Testing completed (on c-c, etc.): c-c, beta
Risk to taking this patch (and alternatives if risky): not risky

Comment 7

[Corey Bryant]

Comment on attachment 9408503 [details]
Bug 1901384 - Add NAME to exported ics calendar. r=#thunderbird-reviewers

[Triage Comment]
Approved for esr128

Comment 8

[Rob Lemley [:rjl]]

The fluent migrations in bug 1893758 make merging this patch rather difficult. I suggest uplifting that bug plus its regression fixes to avoid uplift problems for the next 12 months.

Comment 9

[Alessandro Castellani [:aleca]]

I'm okay with uplifting those if you think they're safe.

Comment 10

[Daniel Darnell [:dandarnell]]

This patch doesn't apply cleanly to ESR 128 and can't be uplifted in it's current state. Could we get an ESR-specific version?

Disregard my previous comment. Should bug 1893758 be uplifted as well to make this patch merge cleanly? There is no uplift requests on bug 1893758.

Comment 11

[Magnus Melin [:mkmelin]]

Bug 1893758 had a ton of regressions do I'd not be too comfortable uplifting that.
If needed for other bugs, maybe, but certainly not for this one.

Comment 12

[Corey Bryant]

Comment on attachment 9408503 [details]
Bug 1901384 - Add NAME to exported ics calendar. r=#thunderbird-reviewers

[Triage Comment]
It seems this will be too risky to uplift to 128esr due to details mentioned in comment 11