{"action":"created","issue":{"url":"https://api.github.com/repos/anuditverma/org.civicrm.osdi/issues/5","labels_url":"https://api.github.com/repos/anuditverma/org.civicrm.osdi/issues/5/labels{/name}","comments_url":"https://api.github.com/repos/anuditverma/org.civicrm.osdi/issues/5/comments","events_url":"https://api.github.com/repos/anuditverma/org.civicrm.osdi/issues/5/events","html_url":"https://github.com/anuditverma/org.civicrm.osdi/issues/5","id":99681707,"number":5,"title":"/people endpoint: don't replicate fields as custom fields","user":{"login":"j-ro","id":1058998,"avatar_url":"https://avatars.githubusercontent.com/u/1058998?v=3","gravatar_id":"","url":"https://api.github.com/users/j-ro","html_url":"https://github.com/j-ro","followers_url":"https://api.github.com/users/j-ro/followers","following_url":"https://api.github.com/users/j-ro/following{/other_user}","gists_url":"https://api.github.com/users/j-ro/gists{/gist_id}","starred_url":"https://api.github.com/users/j-ro/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/j-ro/subscriptions","organizations_url":"https://api.github.com/users/j-ro/orgs","repos_url":"https://api.github.com/users/j-ro/repos","events_url":"https://api.github.com/users/j-ro/events{/privacy}","received_events_url":"https://api.github.com/users/j-ro/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":14,"created_at":"2015-08-07T16:22:31Z","updated_at":"2015-08-20T18:43:00Z","closed_at":"2015-08-20T18:43:00Z","body":"Right now, I'm getting resources on the test endpoint like this:\r\n\r\n```\r\n{\r\n        \"given_name\": \"\",\r\n        \"family_name\": \"\",\r\n        \"email_addresses\": {\r\n          \"primary\": true,\r\n          \"address\": \"fixme.domainemail@example.org\"\r\n        },\r\n        \"identifiers\": [\r\n          \"osdi-person-[0]\"\r\n        ],\r\n        \"id\": \"1\",\r\n        \"created_date\": \"2015-07-03 14:19:10\",\r\n        \"modified_date\": \"2015/08/08\",\r\n        \"custom_fields\": {\r\n          \"email\": \"fixme.domainemail@example.org\",\r\n          \"full_name\": \" \",\r\n          \"event_code\": \"xx\",\r\n          \"address\": \"\",\r\n          \"zip\": null,\r\n          \"pledge\": \"2\"\r\n        },\r\n        \"postal_addresses\": [\r\n          {\r\n            \"address_lines\": [\r\n              null\r\n            ],\r\n            \"postal_code\": null,\r\n            \"address_status\": \"Verified/Not Verified\",\r\n            \"primary\": \"True/False\"\r\n          }\r\n        ],\r\n        \"phone_numbers\": [\r\n          {\r\n            \"number\": null\r\n          }\r\n        ],\r\n        \"_embedded\": {\r\n          \"osdi:tags\": []\r\n        },\r\n        \"_links\": {\r\n          \"self\": {\r\n            \"href\": \"http://api.opensupporter.org/api/v1/people/X\"\r\n          },\r\n          \"addresses\": {\r\n            \"href\": \"http://api.opensupporter.org/api/v1/people/X/addresses\"\r\n          },\r\n          \"question_answers\": {\r\n            \"href\": \"http://api.opensupporter.org/api/v1/people/X/question_answers\"\r\n          },\r\n          \"osdi-tags\": {\r\n            \"href\": \"http://api.opensupporter.org/api/v1/people/X/tags\"\r\n          }\r\n        }\r\n      }\r\n```\r\n\r\nSo email has been made into a custom field, so has address, etc...\r\n\r\nCustom fields should only contain custom field data, rather than duplications of data that already exists elsewhere in the resource."},"comment":{"url":"https://api.github.com/repos/anuditverma/org.civicrm.osdi/issues/comments/133112848","html_url":"https://github.com/anuditverma/org.civicrm.osdi/issues/5#issuecomment-133112848","issue_url":"https://api.github.com/repos/anuditverma/org.civicrm.osdi/issues/5","id":133112848,"user":{"login":"anuditverma","id":7934882,"avatar_url":"https://avatars.githubusercontent.com/u/7934882?v=3","gravatar_id":"","url":"https://api.github.com/users/anuditverma","html_url":"https://github.com/anuditverma","followers_url":"https://api.github.com/users/anuditverma/followers","following_url":"https://api.github.com/users/anuditverma/following{/other_user}","gists_url":"https://api.github.com/users/anuditverma/gists{/gist_id}","starred_url":"https://api.github.com/users/anuditverma/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/anuditverma/subscriptions","organizations_url":"https://api.github.com/users/anuditverma/orgs","repos_url":"https://api.github.com/users/anuditverma/repos","events_url":"https://api.github.com/users/anuditverma/events{/privacy}","received_events_url":"https://api.github.com/users/anuditverma/received_events","type":"User","site_admin":false},"created_at":"2015-08-20T18:43:00Z","updated_at":"2015-08-20T18:43:00Z","body":"This is one fixed, I have removed the fields from custom fields' array to prevent the duplication.. Fixes : https://github.com/anuditverma/org.civicrm.osdi/commit/135aed9f7ef28aff73ef43d11119aa4e4b3aee07 & https://github.com/anuditverma/org.civicrm.osdi/commit/cee48bdff6b62766d176a4e7aa6b4e5a96c9a04f"}}