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

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

Successfully merging this pull request may close these issues.

2 participants