{"action":"created","issue":{"url":"https://api.github.com/repos/tobydragon/wizard/issues/19","labels_url":"https://api.github.com/repos/tobydragon/wizard/issues/19/labels{/name}","comments_url":"https://api.github.com/repos/tobydragon/wizard/issues/19/comments","events_url":"https://api.github.com/repos/tobydragon/wizard/issues/19/events","html_url":"https://github.com/tobydragon/wizard/issues/19","id":87352332,"number":19,"title":"System should allow optional fields for cfActions","user":{"login":"tobydragon","id":6741052,"avatar_url":"https://avatars.githubusercontent.com/u/6741052?v=3","gravatar_id":"","url":"https://api.github.com/users/tobydragon","html_url":"https://github.com/tobydragon","followers_url":"https://api.github.com/users/tobydragon/followers","following_url":"https://api.github.com/users/tobydragon/following{/other_user}","gists_url":"https://api.github.com/users/tobydragon/gists{/gist_id}","starred_url":"https://api.github.com/users/tobydragon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tobydragon/subscriptions","organizations_url":"https://api.github.com/users/tobydragon/orgs","repos_url":"https://api.github.com/users/tobydragon/repos","events_url":"https://api.github.com/users/tobydragon/events{/privacy}","received_events_url":"https://api.github.com/users/tobydragon/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"cwormsl2","id":6827787,"avatar_url":"https://avatars.githubusercontent.com/u/6827787?v=3","gravatar_id":"","url":"https://api.github.com/users/cwormsl2","html_url":"https://github.com/cwormsl2","followers_url":"https://api.github.com/users/cwormsl2/followers","following_url":"https://api.github.com/users/cwormsl2/following{/other_user}","gists_url":"https://api.github.com/users/cwormsl2/gists{/gist_id}","starred_url":"https://api.github.com/users/cwormsl2/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cwormsl2/subscriptions","organizations_url":"https://api.github.com/users/cwormsl2/orgs","repos_url":"https://api.github.com/users/cwormsl2/repos","events_url":"https://api.github.com/users/cwormsl2/events{/privacy}","received_events_url":"https://api.github.com/users/cwormsl2/received_events","type":"User","site_admin":false},"milestone":null,"comments":1,"created_at":"2015-06-11T13:45:15Z","updated_at":"2015-06-22T19:18:12Z","closed_at":null,"body":"Not every analysis cfAction should be required to have all fields, since not all are applicable to all analysis, and this invites backwards compatibility.  We need to understand what happens when a field is missing, and consider how to implement it during creation.\r\n\r\nFor example, in perUserPerProblem, isCorrect is defaulted to true for non-assessable items (as stated in perUserAllProblems constructor.  HOwever, this is not correct, the isCorrect field should not exist for these items, as something not assessable should be neither correct nor incorrect."},"comment":{"url":"https://api.github.com/repos/tobydragon/wizard/issues/comments/114225793","html_url":"https://github.com/tobydragon/wizard/issues/19#issuecomment-114225793","issue_url":"https://api.github.com/repos/tobydragon/wizard/issues/19","id":114225793,"user":{"login":"cwormsl2","id":6827787,"avatar_url":"https://avatars.githubusercontent.com/u/6827787?v=3","gravatar_id":"","url":"https://api.github.com/users/cwormsl2","html_url":"https://github.com/cwormsl2","followers_url":"https://api.github.com/users/cwormsl2/followers","following_url":"https://api.github.com/users/cwormsl2/following{/other_user}","gists_url":"https://api.github.com/users/cwormsl2/gists{/gist_id}","starred_url":"https://api.github.com/users/cwormsl2/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cwormsl2/subscriptions","organizations_url":"https://api.github.com/users/cwormsl2/orgs","repos_url":"https://api.github.com/users/cwormsl2/repos","events_url":"https://api.github.com/users/cwormsl2/events{/privacy}","received_events_url":"https://api.github.com/users/cwormsl2/received_events","type":"User","site_admin":false},"created_at":"2015-06-22T19:18:12Z","updated_at":"2015-06-22T19:18:12Z","body":"I created the separate assessable classes for PerUserPerProblem and AllUsersPerProblems which was sparked by this issue but I don't know if that solves what you were initially thinking when you created this. "}}