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

Bugfixes in conflict resolution #23

Open
wants to merge 114 commits into
base: master
Choose a base branch
from
Open

Conversation

hmshood
Copy link

@hmshood hmshood commented Feb 11, 2011

These are some bug fixes for the conflict detection, especially with relation to one person being at two events at the same time.

… properly submit all parts of a form

(cherry picked from commit cf566d6)
fix attachment url on schedule event page
hmshood and others added 30 commits September 10, 2012 23:48
Reverting rails version to 2.3.5 as later versions generate many deprecation warnings
xCal is a direct port of iCal to XML.
iCal only allows one of dtend and duration to be specified.
The non-normative relax-ng schema in the relevant rfc reflects this.

I removed the duration and not the dtend, as parsers have to support the
datetime format anyways and there are parsers without support for the
duration format.

xCal [rfc 6321](https://tools.ietf.org/html/rfc6321#page-32):

    ```
    (property-dtend | property-duration)? &
    ```

iCal [rfc 5545](https://tools.ietf.org/html/rfc5545#page52):

    ```
    ; Either 'dtend' or 'duration' MAY appear in
    ; a 'eventprop', but 'dtend' and 'duration'
    ; MUST NOT occur in the same 'eventprop'.
    ;
    dtend / duration /
    ```
There are iCal parsers without support for duration values.
As dtend has the same format as dtstart, parsers which don't support it
have no chance anyways.
Corrections to formats of calendar exports for standards compliance
remove packages which should be installed as gems, update some instructions
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.

6 participants