{"action":"created","issue":{"url":"https://api.github.com/repos/BitcoinUnlimited/BitcoinUnlimited/issues/1236","repository_url":"https://api.github.com/repos/BitcoinUnlimited/BitcoinUnlimited","labels_url":"https://api.github.com/repos/BitcoinUnlimited/BitcoinUnlimited/issues/1236/labels{/name}","comments_url":"https://api.github.com/repos/BitcoinUnlimited/BitcoinUnlimited/issues/1236/comments","events_url":"https://api.github.com/repos/BitcoinUnlimited/BitcoinUnlimited/issues/1236/events","html_url":"https://github.com/BitcoinUnlimited/BitcoinUnlimited/pull/1236","id":346719078,"node_id":"MDExOlB1bGxSZXF1ZXN0MjA1NTE1MzYw","number":1236,"title":"[WIP] Extended version message (xversion)","user":{"login":"awemany","id":13838274,"node_id":"MDQ6VXNlcjEzODM4Mjc0","avatar_url":"https://avatars3.githubusercontent.com/u/13838274?v=4","gravatar_id":"","url":"https://api.github.com/users/awemany","html_url":"https://github.com/awemany","followers_url":"https://api.github.com/users/awemany/followers","following_url":"https://api.github.com/users/awemany/following{/other_user}","gists_url":"https://api.github.com/users/awemany/gists{/gist_id}","starred_url":"https://api.github.com/users/awemany/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/awemany/subscriptions","organizations_url":"https://api.github.com/users/awemany/orgs","repos_url":"https://api.github.com/users/awemany/repos","events_url":"https://api.github.com/users/awemany/events{/privacy}","received_events_url":"https://api.github.com/users/awemany/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":18,"created_at":"2018-08-01T18:37:21Z","updated_at":"2018-10-11T21:02:30Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/BitcoinUnlimited/BitcoinUnlimited/pulls/1236","html_url":"https://github.com/BitcoinUnlimited/BitcoinUnlimited/pull/1236","diff_url":"https://github.com/BitcoinUnlimited/BitcoinUnlimited/pull/1236.diff","patch_url":"https://github.com/BitcoinUnlimited/BitcoinUnlimited/pull/1236.patch"},"body":""},"comment":{"url":"https://api.github.com/repos/BitcoinUnlimited/BitcoinUnlimited/issues/comments/429117472","html_url":"https://github.com/BitcoinUnlimited/BitcoinUnlimited/pull/1236#issuecomment-429117472","issue_url":"https://api.github.com/repos/BitcoinUnlimited/BitcoinUnlimited/issues/1236","id":429117472,"node_id":"MDEyOklzc3VlQ29tbWVudDQyOTExNzQ3Mg==","user":{"login":"deadalnix","id":854121,"node_id":"MDQ6VXNlcjg1NDEyMQ==","avatar_url":"https://avatars1.githubusercontent.com/u/854121?v=4","gravatar_id":"","url":"https://api.github.com/users/deadalnix","html_url":"https://github.com/deadalnix","followers_url":"https://api.github.com/users/deadalnix/followers","following_url":"https://api.github.com/users/deadalnix/following{/other_user}","gists_url":"https://api.github.com/users/deadalnix/gists{/gist_id}","starred_url":"https://api.github.com/users/deadalnix/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/deadalnix/subscriptions","organizations_url":"https://api.github.com/users/deadalnix/orgs","repos_url":"https://api.github.com/users/deadalnix/repos","events_url":"https://api.github.com/users/deadalnix/events{/privacy}","received_events_url":"https://api.github.com/users/deadalnix/received_events","type":"User","site_admin":false},"created_at":"2018-10-11T21:02:30Z","updated_at":"2018-10-11T21:02:30Z","author_association":"COLLABORATOR","body":"So, I was thinking about this. The general idea is great, however, I think we should simply use univalue for the metadata. Here is my reasoning:\r\n1/ We don't really need this message to be very compact. It's only sent once. So optimizing binary format is not a very proposition.\r\n2/ We'd get human readable metadata. I'm sure this can be helpful. Currently, no visualization of the data is possible unless the visualizer knows about all the tags, which defeat the purpose.\r\n3/ Univalue is used for the RPC, so it is widely available in existing implementations.\r\n4/ It give us a structure that can be extended all the way down. Currently every extension will have to define how to interpret the content of its binary blob, and you can bet most will not plan to do it in an extensible manner."}}
{"actor":{"display_login":"deadalnix"}}