-
Notifications
You must be signed in to change notification settings - Fork 727
Added resourceWatcher #354
base: master
Are you sure you want to change the base?
Conversation
Resources are supported and work with FullCalendar Scheduler Plugin.
@@ -98,6 +109,7 @@ angular.module('ui.calendar', []) | |||
this.changeWatcher = function(arraySource, tokenFn) { | |||
var self; | |||
var getTokens = function() { | |||
// console.log(arraySource); | |||
var array = angular.isFunction(arraySource) ? arraySource() : arraySource; | |||
var result = [], token, el; | |||
for (var i = 0, n = array.length; i < n; i++) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Travis fails here since test files where not updated. Therefore scope.resources
is undefined
.
I believe line 238 should be modified (see comment there)
@gusreyes01 Thanks for your PR. I am looking forward having this merged into master :-) |
Hey, just commenting here to see if the Scheduler module will be compatible with Angular? |
Did you get scheduler working with uiCalender directive. Thanks |
Yes its working great
…On Mon, 14 Aug 2017 at 00:20, Babugoli ***@***.***> wrote:
Hi @antoinepairet <https://github.com/antoinepairet>
Did you get scheduler working with uiCalender directive.
Thanks
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#354 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AE_vQOLKpZJlu2_DQiAzzO1lTuQmXL_gks5sX3bEgaJpZM4GtjaY>
.
|
it doesnt work for me, i used final changes from raw source, do you mind helping or share sample you using with calendar.js file here is how I'm using $scope.uiConfig = {
$scope.events = [
I do find error as b.class is not a constructor |
thanks @antoinepairet works for me too. appreciate for getting back |
Hello |
Resources are supported and work with FullCalendar Scheduler Plugin.