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

can we make 'method' a VCALENDAR property #9

Open
maxwellutd opened this issue Jun 5, 2015 · 4 comments
Open

can we make 'method' a VCALENDAR property #9

maxwellutd opened this issue Jun 5, 2015 · 4 comments

Comments

@maxwellutd
Copy link

Had issues with getting the calendar invites to appear in the email with RSVP actions. It turned out that
"method: REQUEST" part shouldn't be under VEVENT but VCALENDAR.

@shanebo
Copy link
Owner

shanebo commented Jun 8, 2015

Did you test this on multiple platforms and email clients?

On Jun 4, 2015, at 9:29 PM, maxwellutd notifications@github.com wrote:

Had issues with getting the calendar invites to appear in the email with RSVP actions. It turned out that
"method: REQUEST" part shouldn't be under VEVENT but VCALENDAR.


Reply to this email directly or view it on GitHub #9.

@maxwellutd
Copy link
Author

Yeah i did. Every example that i went through said it needed to be under
the VCALENDAR. That one change made it work on multiple platforms for me.

On Tue, Jun 9, 2015 at 3:32 AM, shanebo notifications@github.com wrote:

Did you test this on multiple platforms and email clients?

On Jun 4, 2015, at 9:29 PM, maxwellutd notifications@github.com wrote:

Had issues with getting the calendar invites to appear in the email with
RSVP actions. It turned out that
"method: REQUEST" part shouldn't be under VEVENT but VCALENDAR.


Reply to this email directly or view it on GitHub <
https://github.com/shanebo/icalevent/issues/9>.


Reply to this email directly or view it on GitHub
#9 (comment).

@shanebo
Copy link
Owner

shanebo commented Jul 2, 2015

@maxwellutd great. Please submit a pull request with the change and I'll review it. Thanks.

@wearhere
Copy link

wearhere commented Feb 8, 2016

This is fixed by @sourcehunter in #7.

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

No branches or pull requests

3 participants