{"action":"created","issue":{"url":"https://api.github.com/repos/bigchaindb/bigchaindb-common/issues/39","repository_url":"https://api.github.com/repos/bigchaindb/bigchaindb-common","labels_url":"https://api.github.com/repos/bigchaindb/bigchaindb-common/issues/39/labels{/name}","comments_url":"https://api.github.com/repos/bigchaindb/bigchaindb-common/issues/39/comments","events_url":"https://api.github.com/repos/bigchaindb/bigchaindb-common/issues/39/events","html_url":"https://github.com/bigchaindb/bigchaindb-common/issues/39","id":182043587,"number":39,"title":"Rename `data_id` in the Metadata and Asset models","user":{"login":"r-marques","id":1793187,"avatar_url":"https://avatars.githubusercontent.com/u/1793187?v=3","gravatar_id":"","url":"https://api.github.com/users/r-marques","html_url":"https://github.com/r-marques","followers_url":"https://api.github.com/users/r-marques/followers","following_url":"https://api.github.com/users/r-marques/following{/other_user}","gists_url":"https://api.github.com/users/r-marques/gists{/gist_id}","starred_url":"https://api.github.com/users/r-marques/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/r-marques/subscriptions","organizations_url":"https://api.github.com/users/r-marques/orgs","repos_url":"https://api.github.com/users/r-marques/repos","events_url":"https://api.github.com/users/r-marques/events{/privacy}","received_events_url":"https://api.github.com/users/r-marques/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-10-10T15:25:21Z","updated_at":"2016-10-17T09:44:36Z","closed_at":null,"body":"I find the name `data_id` confusing in both the `Metadata` and `Asset` model.\r\nSince they are already inside a namespace I propose to rename `data_id` to `id`.\r\nThis way we have:\r\n- `Transaction.id`\r\n- `Transaction.metadata.id`\r\n- `Transaction.asset.id`"},"comment":{"url":"https://api.github.com/repos/bigchaindb/bigchaindb-common/issues/comments/254161645","html_url":"https://github.com/bigchaindb/bigchaindb-common/issues/39#issuecomment-254161645","issue_url":"https://api.github.com/repos/bigchaindb/bigchaindb-common/issues/39","id":254161645,"user":{"login":"sbellem","id":125458,"avatar_url":"https://avatars.githubusercontent.com/u/125458?v=3","gravatar_id":"","url":"https://api.github.com/users/sbellem","html_url":"https://github.com/sbellem","followers_url":"https://api.github.com/users/sbellem/followers","following_url":"https://api.github.com/users/sbellem/following{/other_user}","gists_url":"https://api.github.com/users/sbellem/gists{/gist_id}","starred_url":"https://api.github.com/users/sbellem/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sbellem/subscriptions","organizations_url":"https://api.github.com/users/sbellem/orgs","repos_url":"https://api.github.com/users/sbellem/repos","events_url":"https://api.github.com/users/sbellem/events{/privacy}","received_events_url":"https://api.github.com/users/sbellem/received_events","type":"User","site_admin":false},"created_at":"2016-10-17T09:44:36Z","updated_at":"2016-10-17T09:44:36Z","body":"As an aside, we may want to consider using a different name than `id` since it is Python built-in function. It would leave no chance for ambiguity to arise."}}
{"actor":{"display_login":"sbellem"}}