{"action":"created","issue":{"url":"https://api.github.com/repos/vaadin/framework/issues/8343","repository_url":"https://api.github.com/repos/vaadin/framework","labels_url":"https://api.github.com/repos/vaadin/framework/issues/8343/labels{/name}","comments_url":"https://api.github.com/repos/vaadin/framework/issues/8343/comments","events_url":"https://api.github.com/repos/vaadin/framework/issues/8343/events","html_url":"https://github.com/vaadin/framework/pull/8343","id":203364388,"number":8343,"title":"Improve Grid extendability","user":{"login":"wbadam","id":15823229,"avatar_url":"https://avatars.githubusercontent.com/u/15823229?v=3","gravatar_id":"","url":"https://api.github.com/users/wbadam","html_url":"https://github.com/wbadam","followers_url":"https://api.github.com/users/wbadam/followers","following_url":"https://api.github.com/users/wbadam/following{/other_user}","gists_url":"https://api.github.com/users/wbadam/gists{/gist_id}","starred_url":"https://api.github.com/users/wbadam/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/wbadam/subscriptions","organizations_url":"https://api.github.com/users/wbadam/orgs","repos_url":"https://api.github.com/users/wbadam/repos","events_url":"https://api.github.com/users/wbadam/events{/privacy}","received_events_url":"https://api.github.com/users/wbadam/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2017-01-26T12:52:21Z","updated_at":"2017-01-27T08:59:21Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/vaadin/framework/pulls/8343","html_url":"https://github.com/vaadin/framework/pull/8343","diff_url":"https://github.com/vaadin/framework/pull/8343.diff","patch_url":"https://github.com/vaadin/framework/pull/8343.patch"},"body":"Find GridState field type using getField() instead of getDeclaredField() so that the field could be found in superclass as well.\r\n\r\nTests for Grid column reordering and Grid column visibility should prove if this patch works.\r\n\r\n Fixes #8342\r\n\r\n<!-- Reviewable:start -->\r\n---\r\nThis change is [<img src=\"https://reviewable.io/review_button.svg\" height=\"34\" align=\"absmiddle\" alt=\"Reviewable\"/>](https://reviewable.io/reviews/vaadin/framework/8343)\r\n<!-- Reviewable:end -->\r\n"},"comment":{"url":"https://api.github.com/repos/vaadin/framework/issues/comments/275618066","html_url":"https://github.com/vaadin/framework/pull/8343#issuecomment-275618066","issue_url":"https://api.github.com/repos/vaadin/framework/issues/8343","id":275618066,"user":{"login":"denis-anisimov","id":11425472,"avatar_url":"https://avatars.githubusercontent.com/u/11425472?v=3","gravatar_id":"","url":"https://api.github.com/users/denis-anisimov","html_url":"https://github.com/denis-anisimov","followers_url":"https://api.github.com/users/denis-anisimov/followers","following_url":"https://api.github.com/users/denis-anisimov/following{/other_user}","gists_url":"https://api.github.com/users/denis-anisimov/gists{/gist_id}","starred_url":"https://api.github.com/users/denis-anisimov/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/denis-anisimov/subscriptions","organizations_url":"https://api.github.com/users/denis-anisimov/orgs","repos_url":"https://api.github.com/users/denis-anisimov/repos","events_url":"https://api.github.com/users/denis-anisimov/events{/privacy}","received_events_url":"https://api.github.com/users/denis-anisimov/received_events","type":"User","site_admin":false},"created_at":"2017-01-27T08:59:21Z","updated_at":"2017-01-27T08:59:21Z","body":"It actually fixes the original bug in the Grid: the code is written in the way to support extended Grid with a derived state but in reality Grid state won't be handled properly in that case.\n\n---\n\nReview status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.\n\n---\n\n\n\n*Comments from [Reviewable](https://reviewable.io:443/reviews/vaadin/framework/8343#-:-KbU6Kttc_rM6GYbSdeI:bjq0tvi)*\n<!-- Sent from Reviewable.io -->\n"}}
{"actor":{"display_login":"denis-anisimov"}}