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

Register calendar before initialize #382

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

Conversation

neilvana
Copy link

Client code can define hooks for fullCalendar in the options that can be called during initialization. For example if a caller specifies "viewRender" and inside viewRender they would like to access the calendar object, this isn't possible since the uiCalendarConfig is not yet configured. By adding the new calendar to uiCalendarConfig before calling fullCalendar(options) it allows clients to define these functions.

Client code can define hooks for fullCalendar in the options that can be called during initialization.  For example if a caller specifies "viewRender" and inside viewRender they would like to access the calendar object, this isn't possible since the uiCalendarConfig is not yet configured.  By adding the new calendar to uiCalendarConfig before calling fullCalendar(options) this allows this functionality.
@neilvana neilvana closed this Feb 14, 2016
@neilvana neilvana reopened this Feb 14, 2016
@neilvana neilvana closed this Feb 14, 2016
@neilvana neilvana reopened this Feb 14, 2016
@neilvana neilvana closed this Feb 17, 2016
@neilvana neilvana reopened this Feb 17, 2016
@neilvana neilvana closed this Mar 10, 2016
@neilvana neilvana reopened this Mar 10, 2016
@neilvana
Copy link
Author

I also had to update the angular dependency since 1.4.1 didn't seem to be available anymore.

@tetyanochka
Copy link

Hi
I need this change too. Do you have you any idea, when it can be merged to the master?
And how do you think, maybe such change has to be in this method also:

                    eventSourcesWatcher.onAdded = function (source) {
                        if (calendar && calendar.fullCalendar) {
                   (2)     calendar.fullCalendar(options);
                   (1)     if (attrs.calendar) {
                                uiCalendarConfig.calendars[attrs.calendar] = calendar;
                            }
                            calendar.fullCalendar('addEventSource', source);
                            sourcesChanged = true;
                        }
                    };

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