)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"16e79743d400dcd7a57dd49787cb742567e825ed","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Wlodzimierz Borkowski \u003cwlodek.borkowski@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2014-09-23 10:00:06 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"get network_id from self.initial[\u0027network_id\u0027]"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Remove hidden field \u0027network_id\u0027 from form, as in can be"},{"line_number":10,"context_line":"retrieved from neutron api, and is no longer needed in"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da9df570_bcdbac7a","line":7,"updated":"2014-09-26 13:57:45.000000000","message":"I think the main purpose of this change is still to remove read-only network ID field. The summary line should describe the highlight of the change.","commit_id":"20b1b3d07290f062a1f6d67a6d615c01fec22878"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"16e79743d400dcd7a57dd49787cb742567e825ed","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"get network_id from self.initial[\u0027network_id\u0027]"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Remove hidden field \u0027network_id\u0027 from form, as in can be"},{"line_number":10,"context_line":"retrieved from neutron api, and is no longer needed in"},{"line_number":11,"context_line":"clent \u0027create network operation\u0027 as can not be changed."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da9df570_1c84b85c","line":9,"updated":"2014-09-26 13:57:45.000000000","message":"The commit message seems not correct. \"hidden\" field was introduced in patch set 1, The commit message should describe the whole change in this review rather than describing a difference from the previous patch set.","commit_id":"20b1b3d07290f062a1f6d67a6d615c01fec22878"}],"openstack_dashboard/dashboards/project/networks/forms.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"dab041bb43775e667909c314db615cbc0c6f2c26","unresolved":false,"context_lines":[{"line_number":34,"context_line":"class UpdateNetwork(forms.SelfHandlingForm):"},{"line_number":35,"context_line":"    name \u003d forms.CharField(label\u003d_(\"Name\"), required\u003dFalse)"},{"line_number":36,"context_line":"    tenant_id \u003d forms.CharField(widget\u003dforms.HiddenInput)"},{"line_number":37,"context_line":"    network_id \u003d forms.CharField(widget\u003dforms.HiddenInput)"},{"line_number":38,"context_line":"    # TODO(amotoki): make UP/DOWN translatable"},{"line_number":39,"context_line":"    admin_state \u003d forms.ChoiceField(choices\u003d[(True, \u0027UP\u0027), (False, \u0027DOWN\u0027)],"},{"line_number":40,"context_line":"                                    label\u003d_(\"Admin State\"))"}],"source_content_type":"text/x-python","patch_set":1,"id":"da9df570_a7adc78a","line":37,"updated":"2014-09-23 07:13:02.000000000","message":"Actually network_id is no longer needed. network_id posted is used to retrieve network_id passed to api.network.update_network in forms.py, but we can get network_id from self.initial[\u0027network_id\u0027]. Could you clean up the code?","commit_id":"8cb626629bcec8a3dfef2873d700f54f68a4fe42"}]}