Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add support for DESCRIPTION in ICS output #157

Closed
cdubz opened this issue Sep 30, 2022 · 1 comment · Fixed by #158
Closed

Add support for DESCRIPTION in ICS output #157

cdubz opened this issue Sep 30, 2022 · 1 comment · Fixed by #158
Labels
feature request service: ics

Comments

@cdubz
Copy link
Contributor

cdubz commented Sep 30, 2022

What do you want to achieve?

#136 replaced the ICS DESCRIPTION with X-ALT-DESC which is only supported by Microsoft products. This value does not get imported by other calendars (e.g., the Calendar app on macOS).

I think it would be ideal if DESCRIPTION could be re-added and potentially forced to plaintext. From a quick manual test it appears that having both DESCRIPTION and X-ALT-DESC is fine -- in that case Calendar.app uses DESCRIPTION and Outlook uses X-ALT-DESC.
@cdubz cdubz changed the title Add support for plaintext DESCRIPTION Add support for DESCRIPTION in ICS output Sep 30, 2022
@alies-dev
Copy link
Collaborator

hey @cdubz
Agree! The simpler - the better. Can you please help us with a PR? This is a hacktoberfest BTW :)

@cdubz cdubz mentioned this issue Oct 6, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature request service: ics
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add DESCRIPTION to ICS output
2 participants
@alies-dev @cdubz