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

Implement CLASS attribute of ics files #161

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

keinstein
Copy link

This patch implements the CLASS attribute for CalDAV servers. It is completely copy&paste&(search&replace) from the LOCATION attribute.

This is the companion patch to patch b32415dede200746f9bc75c247379a389f76db1e from org-mode.

This allows to control the visibilty of calendar entries. This is necessary to properly synchronize semi-public calendars, for example public and private performances of a band in one calendar.

BTW. I didn't know how to introduce this feature to README.md. LOCATION is mentioned there as unhandled, but both ox-icalendar and org-caldav support it.

@niclasborlin
Copy link
Contributor

@keinstein, you may want the patch indicated by pull request #162. It looks like the addition of the calss attribute triggered a latent bug in the main repo. Pull request #162 contains a fix for that.

@keinstein
Copy link
Author

@niclasborlin Thanks, I checked out your master branch with the additonal patches for personal use and testing. Nevertheless I think #162 is worth to be integrated independently into upstream.

@dakra dakra mentioned this pull request Jun 26, 2019
dakra added a commit to dakra/org-caldav that referenced this pull request Jun 27, 2019
dakra added a commit to dakra/org-caldav that referenced this pull request Jun 27, 2019
@dengste
Copy link
Owner

dengste commented Aug 5, 2019

@keinstein You introduce org-caldav-change-class but it is never called.

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