{"action":"created","issue":{"url":"https://api.github.com/repos/NuCivic/data_workflow/issues/33","labels_url":"https://api.github.com/repos/NuCivic/data_workflow/issues/33/labels{/name}","comments_url":"https://api.github.com/repos/NuCivic/data_workflow/issues/33/comments","events_url":"https://api.github.com/repos/NuCivic/data_workflow/issues/33/events","html_url":"https://github.com/NuCivic/data_workflow/pull/33","id":106253827,"number":33,"title":"Remove field overrides conflicting  healthdata_settings.","user":{"login":"dkinzer","id":444215,"avatar_url":"https://avatars.githubusercontent.com/u/444215?v=3","gravatar_id":"","url":"https://api.github.com/users/dkinzer","html_url":"https://github.com/dkinzer","followers_url":"https://api.github.com/users/dkinzer/followers","following_url":"https://api.github.com/users/dkinzer/following{/other_user}","gists_url":"https://api.github.com/users/dkinzer/gists{/gist_id}","starred_url":"https://api.github.com/users/dkinzer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dkinzer/subscriptions","organizations_url":"https://api.github.com/users/dkinzer/orgs","repos_url":"https://api.github.com/users/dkinzer/repos","events_url":"https://api.github.com/users/dkinzer/events{/privacy}","received_events_url":"https://api.github.com/users/dkinzer/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-09-14T01:07:44Z","updated_at":"2015-09-16T13:51:53Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/NuCivic/data_workflow/pulls/33","html_url":"https://github.com/NuCivic/data_workflow/pull/33","diff_url":"https://github.com/NuCivic/data_workflow/pull/33.diff","patch_url":"https://github.com/NuCivic/data_workflow/pull/33.patch"},"body":"I could not get healthdata features to not be overridden until I made this\nchange:  Features went from 16 overridden features to 8.\n\nRef: https://github.com/nucivic/healthdata/issues/437\n\nAcceptance:\n- [ ] Build rebuild hhs and you should see overrides come down from 16 to 8 or 9."},"comment":{"url":"https://api.github.com/repos/NuCivic/data_workflow/issues/comments/140748177","html_url":"https://github.com/NuCivic/data_workflow/pull/33#issuecomment-140748177","issue_url":"https://api.github.com/repos/NuCivic/data_workflow/issues/33","id":140748177,"user":{"login":"dkinzer","id":444215,"avatar_url":"https://avatars.githubusercontent.com/u/444215?v=3","gravatar_id":"","url":"https://api.github.com/users/dkinzer","html_url":"https://github.com/dkinzer","followers_url":"https://api.github.com/users/dkinzer/followers","following_url":"https://api.github.com/users/dkinzer/following{/other_user}","gists_url":"https://api.github.com/users/dkinzer/gists{/gist_id}","starred_url":"https://api.github.com/users/dkinzer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dkinzer/subscriptions","organizations_url":"https://api.github.com/users/dkinzer/orgs","repos_url":"https://api.github.com/users/dkinzer/repos","events_url":"https://api.github.com/users/dkinzer/events{/privacy}","received_events_url":"https://api.github.com/users/dkinzer/received_events","type":"User","site_admin":false},"created_at":"2015-09-16T13:51:53Z","updated_at":"2015-09-16T13:51:53Z","body":"@rhabbachi Oh, that concerns me quite a bit.  Especially since these changes already got merged into HHS master via  https://github.com/NuCivic/healthdata/pull/447. Did I inadvertently just completely gut data_workflow?   \r\n\r\nAll I did was remove the field overrides that were in conflict with healthdata settings in the gui via:\r\n```\r\n-features[features_override_items][] = variable.node_options_dataset\t\t\r\n-features[features_override_items][] = variable.node_options_resource\t\t\r\n-features[features_overrides][] = variable.node_options_dataset.value|0\t\t\r\n-features[features_overrides][] = variable.node_options_resource.value|0\r\n```\r\n\r\nThen I re exported the feature and the rest happened automatically.  I was assuming that whatever overrides where happening were being taken care of by healthdata_settings.   I looked at the data workflow functionality and everything seemed to be in order.. But perhaps I was reviewing the wrong thing,\r\n\r\nIf this is going to break data_workflow I'll need to revert that change on master..  But this means back to square one with overridden changes.\r\n\r\nPerhaps another option is to remove the conflicts on the helthdata_settings end.\r\n"}}