Skip to content
This repository has been archived by the owner on Sep 8, 2020. It is now read-only.

Fixed event '_id' assignment. #171

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

Conversation

envyN
Copy link

@envyN envyN commented Oct 24, 2014

Fixed event '_id' attribute assignment..
Events with same 'id' required for repeat functionality were being assigned different '_id' earlier.
This resulted in breaking functionality for fullcalendar's repeat event functionality wherein repeated events repositioning is supposed to reposition all events by the same day-difference-delta.

…uncionality were being assigned different '_id' 's earlier.
@envyN
Copy link
Author

envyN commented Oct 24, 2014

If this fix gets accepted it is going to be my first Open-Source contribution! :) Had to learn how to fork and create PRs for contributing to existing great libraries like this. Something new learnt for me today. Cheers.

envyN added 3 commits October 25, 2014 08:44
…n the token. Checked for presence of _i attribute in the event since updation from fullcalendar of the dates to moment dates caused triggering of onChanged event.
@joshkurz
Copy link
Contributor

Hey sorry about the delay on this. I like this PR, could you close and send another one with the same functionality? thanks

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

Successfully merging this pull request may close these issues.

2 participants