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.

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

Fix Outlook 2016 - ics: description shows no html formatting #136

Merged
merged 1 commit into from
Aug 20, 2022

Conversation

karthikbodu
Copy link

Committer: Karthik Kumar Bodu karthik.mvgr@gmail.com

Context and Purposes

Problem : The HTML formatting is lost when DESCRIPTION is used in the ICS file.

Reference : https://answers.microsoft.com/en-us/outlook_com/forum/all/outlook-2016-ics-description-shows-no-html/08d06cba-bfe4-4757-a052-adab64ea75a2?page=1

Solution : Using X-ALT-DESC;FMTTYPE=text/html will resolve the problem and will render the description if there are HTML tags.

Committer: Karthik Kumar Bodu <karthik.mvgr@gmail.com>
@spatie-bot
Copy link

Dear contributor,

because this pull request seems to be inactive for quite some time now, I've automatically closed it. If you feel this pull request deserves some attention from my human colleagues feel free to reopen it.

@spatie-bot spatie-bot closed this Mar 16, 2022
@alies-dev alies-dev reopened this Mar 17, 2022
@spatie-bot
Copy link

Dear contributor,

because this pull request seems to be inactive for quite some time now, I've automatically closed it. If you feel this pull request deserves some attention from my human colleagues feel free to reopen it.

@spatie-bot spatie-bot closed this Jul 18, 2022
@alies-dev alies-dev reopened this Jul 18, 2022
Copy link
Collaborator

@alies-dev alies-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍
it looks good! I'm not able to properly test it, as some other PR (this is a big problem of maintaining this package). Merging it now and will listen for feedback from the community, I don't want to block this PR anymore. Sorry for the delay

@alies-dev alies-dev merged commit 72888da into spatie:master Aug 20, 2022
@alies-dev
Copy link
Collaborator

Sorry, I have to revert these changes. Details you can find in #158 (review)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants