{"action":"created","issue":{"url":"https://api.github.com/repos/aldeed/meteor-tabular/issues/117","labels_url":"https://api.github.com/repos/aldeed/meteor-tabular/issues/117/labels{/name}","comments_url":"https://api.github.com/repos/aldeed/meteor-tabular/issues/117/comments","events_url":"https://api.github.com/repos/aldeed/meteor-tabular/issues/117/events","html_url":"https://github.com/aldeed/meteor-tabular/pull/117","id":64941643,"number":117,"title":"Enabling field changes to be made reactively","user":{"login":"dovrosenberg","id":4704591,"avatar_url":"https://avatars.githubusercontent.com/u/4704591?v=3","gravatar_id":"","url":"https://api.github.com/users/dovrosenberg","html_url":"https://github.com/dovrosenberg","followers_url":"https://api.github.com/users/dovrosenberg/followers","following_url":"https://api.github.com/users/dovrosenberg/following{/other_user}","gists_url":"https://api.github.com/users/dovrosenberg/gists{/gist_id}","starred_url":"https://api.github.com/users/dovrosenberg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dovrosenberg/subscriptions","organizations_url":"https://api.github.com/users/dovrosenberg/orgs","repos_url":"https://api.github.com/users/dovrosenberg/repos","events_url":"https://api.github.com/users/dovrosenberg/events{/privacy}","received_events_url":"https://api.github.com/users/dovrosenberg/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-03-28T13:15:19Z","updated_at":"2015-05-05T13:45:53Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/aldeed/meteor-tabular/pulls/117","html_url":"https://github.com/aldeed/meteor-tabular/pull/117","diff_url":"https://github.com/aldeed/meteor-tabular/pull/117.diff","patch_url":"https://github.com/aldeed/meteor-tabular/pull/117.patch"},"body":"With respect to issue #85.\r\n\r\nI made the following changes to allow changes to fields to refresh the table reactively...\r\n\r\n1) changed tabular.fields to be reactive - this is used to ensure a new subscription is generated when it changes\r\n\r\n2) I now regenerate ajaxOptions each time a new DataTable is created (because otherwise it gets contaminated with old aoColumns by the prior DataTable)\r\n\r\n3) Commented out the block at new line 63 that was preventing options from being updated if the table name hasn't changed... this piece makes me nervous about side effects - that reload is probably required in some circumstances... it may need to be replaced, but further down in that autorun.  \r\n\r\nEDIT: I missed there's already a reload further down, so #3 probably isn't actually an issue.\r\n\r\nThoughts?\r\n\r\n"},"comment":{"url":"https://api.github.com/repos/aldeed/meteor-tabular/issues/comments/99082422","html_url":"https://github.com/aldeed/meteor-tabular/pull/117#issuecomment-99082422","issue_url":"https://api.github.com/repos/aldeed/meteor-tabular/issues/117","id":99082422,"user":{"login":"joramhoefs","id":560214,"avatar_url":"https://avatars.githubusercontent.com/u/560214?v=3","gravatar_id":"","url":"https://api.github.com/users/joramhoefs","html_url":"https://github.com/joramhoefs","followers_url":"https://api.github.com/users/joramhoefs/followers","following_url":"https://api.github.com/users/joramhoefs/following{/other_user}","gists_url":"https://api.github.com/users/joramhoefs/gists{/gist_id}","starred_url":"https://api.github.com/users/joramhoefs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/joramhoefs/subscriptions","organizations_url":"https://api.github.com/users/joramhoefs/orgs","repos_url":"https://api.github.com/users/joramhoefs/repos","events_url":"https://api.github.com/users/joramhoefs/events{/privacy}","received_events_url":"https://api.github.com/users/joramhoefs/received_events","type":"User","site_admin":false},"created_at":"2015-05-05T13:45:53Z","updated_at":"2015-05-05T13:45:53Z","body":"+1"}}