)]}'
{"manila_ui/dashboards/admin/security_services/templates/security_services/_detail.html":[{"author":{"_account_id":32531,"name":"Dina Saparbaeva","email":"dinas27@outlook.com","username":"disap"},"change_message_id":"21908372b52bd3fd053d0bd08ac6ef1fddc1c999","unresolved":false,"context_lines":[{"line_number":4,"context_line":"\u003cdiv class\u003d\"detail\"\u003e"},{"line_number":5,"context_line":"  \u003chr class\u003d\"header_rule\"\u003e"},{"line_number":6,"context_line":"  \u003cdl class\u003d\"dl-horizontal\"\u003e"},{"line_number":7,"context_line":"    \u003cdt\u003e{% trans \"Type\" %}\u003c/dt\u003e"},{"line_number":8,"context_line":"    \u003cdd\u003e{{ sec_service.type }}\u003c/dd\u003e"},{"line_number":9,"context_line":"    \u003cdt\u003e{% trans \"Name\" %}\u003c/dt\u003e"},{"line_number":10,"context_line":"    \u003cdd\u003e{{ sec_service.name }}\u003c/dd\u003e"}],"source_content_type":"text/html","patch_set":7,"id":"678c2e2d_121be87d","line":7,"updated":"2021-03-12 05:38:50.000000000","message":"Here is Type and name, I will add description here as well in a separate change.","commit_id":"70a932372c58d1f9c80d3bbf747065a0ec324b67"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"ae7b30efcb54f7fd95132451b9c401e62e601392","unresolved":false,"context_lines":[{"line_number":4,"context_line":"\u003cdiv class\u003d\"detail\"\u003e"},{"line_number":5,"context_line":"  \u003chr class\u003d\"header_rule\"\u003e"},{"line_number":6,"context_line":"  \u003cdl class\u003d\"dl-horizontal\"\u003e"},{"line_number":7,"context_line":"    \u003cdt\u003e{% trans \"Type\" %}\u003c/dt\u003e"},{"line_number":8,"context_line":"    \u003cdd\u003e{{ sec_service.type }}\u003c/dd\u003e"},{"line_number":9,"context_line":"    \u003cdt\u003e{% trans \"Name\" %}\u003c/dt\u003e"},{"line_number":10,"context_line":"    \u003cdd\u003e{{ sec_service.name }}\u003c/dd\u003e"}],"source_content_type":"text/html","patch_set":7,"id":"00123f55_8107d8a0","line":7,"in_reply_to":"678c2e2d_121be87d","updated":"2021-03-12 08:15:30.000000000","message":"Thanks, was blind 😊","commit_id":"70a932372c58d1f9c80d3bbf747065a0ec324b67"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"1e3faf19ee96dbd66c59edf56fee6b787d5ada16","unresolved":true,"context_lines":[{"line_number":23,"context_line":"\u003cdiv class\u003d\"detail\"\u003e"},{"line_number":24,"context_line":"  \u003ch4\u003e{% trans \"Security Service Details\" %}\u003c/h4\u003e"},{"line_number":25,"context_line":"  \u003chr class\u003d\"header_rule\"\u003e"},{"line_number":26,"context_line":"  \u003cdl class\u003d\"dl-horizontal\"\u003e"},{"line_number":27,"context_line":"    \u003cdt\u003e{% trans \"DNS IP\" %}\u003c/dt\u003e"},{"line_number":28,"context_line":"    \u003cdd\u003e{{ sec_service.dns_ip }}\u003c/dd\u003e"},{"line_number":29,"context_line":"    \u003cdt\u003e{% trans \"Organizational Unit\" %}\u003c/dt\u003e"}],"source_content_type":"text/html","patch_set":7,"id":"9a1b8ffe_0e508c12","line":26,"range":{"start_line":26,"start_character":27,"end_line":26,"end_character":28},"updated":"2021-03-11 18:30:14.000000000","message":"related question, but unrelated to your change - doesn\u0027t it make sense to have Name, description and security service type here?","commit_id":"70a932372c58d1f9c80d3bbf747065a0ec324b67"},{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"dd4a4556abed8f91a0520c44c8fe297d4ec27256","unresolved":true,"context_lines":[{"line_number":23,"context_line":"\u003cdiv class\u003d\"detail\"\u003e"},{"line_number":24,"context_line":"  \u003ch4\u003e{% trans \"Security Service Details\" %}\u003c/h4\u003e"},{"line_number":25,"context_line":"  \u003chr class\u003d\"header_rule\"\u003e"},{"line_number":26,"context_line":"  \u003cdl class\u003d\"dl-horizontal\"\u003e"},{"line_number":27,"context_line":"    \u003cdt\u003e{% trans \"DNS IP\" %}\u003c/dt\u003e"},{"line_number":28,"context_line":"    \u003cdd\u003e{{ sec_service.dns_ip }}\u003c/dd\u003e"},{"line_number":29,"context_line":"    \u003cdt\u003e{% trans \"Organizational Unit\" %}\u003c/dt\u003e"}],"source_content_type":"text/html","patch_set":7,"id":"d1c91c32_0305a053","line":26,"range":{"start_line":26,"start_character":27,"end_line":26,"end_character":28},"in_reply_to":"9a1b8ffe_0e508c12","updated":"2021-03-12 00:21:50.000000000","message":"Not very familiar with how the security services are used, do you think an operator might find that information useful? If so, we can address it in a follow up patch","commit_id":"70a932372c58d1f9c80d3bbf747065a0ec324b67"},{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"2f287fd6659faeb5c930f9cb202e689bf1500c93","unresolved":false,"context_lines":[{"line_number":23,"context_line":"\u003cdiv class\u003d\"detail\"\u003e"},{"line_number":24,"context_line":"  \u003ch4\u003e{% trans \"Security Service Details\" %}\u003c/h4\u003e"},{"line_number":25,"context_line":"  \u003chr class\u003d\"header_rule\"\u003e"},{"line_number":26,"context_line":"  \u003cdl class\u003d\"dl-horizontal\"\u003e"},{"line_number":27,"context_line":"    \u003cdt\u003e{% trans \"DNS IP\" %}\u003c/dt\u003e"},{"line_number":28,"context_line":"    \u003cdd\u003e{{ sec_service.dns_ip }}\u003c/dd\u003e"},{"line_number":29,"context_line":"    \u003cdt\u003e{% trans \"Organizational Unit\" %}\u003c/dt\u003e"}],"source_content_type":"text/html","patch_set":7,"id":"c9144655_b5697e3d","line":26,"range":{"start_line":26,"start_character":27,"end_line":26,"end_character":28},"in_reply_to":"babd9745_2ab36d88","updated":"2021-03-12 14:17:47.000000000","message":"Ouch, missed that as well. Thanks for addressing it.","commit_id":"70a932372c58d1f9c80d3bbf747065a0ec324b67"},{"author":{"_account_id":32531,"name":"Dina Saparbaeva","email":"dinas27@outlook.com","username":"disap"},"change_message_id":"21908372b52bd3fd053d0bd08ac6ef1fddc1c999","unresolved":false,"context_lines":[{"line_number":23,"context_line":"\u003cdiv class\u003d\"detail\"\u003e"},{"line_number":24,"context_line":"  \u003ch4\u003e{% trans \"Security Service Details\" %}\u003c/h4\u003e"},{"line_number":25,"context_line":"  \u003chr class\u003d\"header_rule\"\u003e"},{"line_number":26,"context_line":"  \u003cdl class\u003d\"dl-horizontal\"\u003e"},{"line_number":27,"context_line":"    \u003cdt\u003e{% trans \"DNS IP\" %}\u003c/dt\u003e"},{"line_number":28,"context_line":"    \u003cdd\u003e{{ sec_service.dns_ip }}\u003c/dd\u003e"},{"line_number":29,"context_line":"    \u003cdt\u003e{% trans \"Organizational Unit\" %}\u003c/dt\u003e"}],"source_content_type":"text/html","patch_set":7,"id":"df735187_8d0df2b6","line":26,"range":{"start_line":26,"start_character":27,"end_line":26,"end_character":28},"in_reply_to":"d1c91c32_0305a053","updated":"2021-03-12 05:38:50.000000000","message":"Type, Name is above on line #7, 9. I will add description, I think it is better to provide complete details. thanks.","commit_id":"70a932372c58d1f9c80d3bbf747065a0ec324b67"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"ae7b30efcb54f7fd95132451b9c401e62e601392","unresolved":false,"context_lines":[{"line_number":23,"context_line":"\u003cdiv class\u003d\"detail\"\u003e"},{"line_number":24,"context_line":"  \u003ch4\u003e{% trans \"Security Service Details\" %}\u003c/h4\u003e"},{"line_number":25,"context_line":"  \u003chr class\u003d\"header_rule\"\u003e"},{"line_number":26,"context_line":"  \u003cdl class\u003d\"dl-horizontal\"\u003e"},{"line_number":27,"context_line":"    \u003cdt\u003e{% trans \"DNS IP\" %}\u003c/dt\u003e"},{"line_number":28,"context_line":"    \u003cdd\u003e{{ sec_service.dns_ip }}\u003c/dd\u003e"},{"line_number":29,"context_line":"    \u003cdt\u003e{% trans \"Organizational Unit\" %}\u003c/dt\u003e"}],"source_content_type":"text/html","patch_set":7,"id":"babd9745_2ab36d88","line":26,"range":{"start_line":26,"start_character":27,"end_line":26,"end_character":28},"in_reply_to":"df735187_8d0df2b6","updated":"2021-03-12 08:15:30.000000000","message":"Nope, I didn\u0027t look at the hidden piece here. This looks good!","commit_id":"70a932372c58d1f9c80d3bbf747065a0ec324b67"}],"manila_ui/dashboards/project/security_services/forms.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"1e3faf19ee96dbd66c59edf56fee6b787d5ada16","unresolved":true,"context_lines":[{"line_number":41,"context_line":"    confirm_password \u003d forms.CharField("},{"line_number":42,"context_line":"        label\u003d_(\"Confirm Password\"),"},{"line_number":43,"context_line":"        widget\u003dforms.PasswordInput(render_value\u003dFalse),"},{"line_number":44,"context_line":"        required\u003dFalse)"},{"line_number":45,"context_line":"    type \u003d forms.ChoiceField(choices\u003d((\"\", \"\"),"},{"line_number":46,"context_line":"                                      (\"active_directory\", \"Active Directory\"),"},{"line_number":47,"context_line":"                                      (\"ldap\", \"LDAP\"),"}],"source_content_type":"text/x-python","patch_set":7,"id":"e9877646_29cc8423","line":44,"range":{"start_line":44,"start_character":8,"end_line":44,"end_character":22},"updated":"2021-03-11 18:30:14.000000000","message":"why?","commit_id":"70a932372c58d1f9c80d3bbf747065a0ec324b67"},{"author":{"_account_id":32531,"name":"Dina Saparbaeva","email":"dinas27@outlook.com","username":"disap"},"change_message_id":"3ea8bd409cbcae86b6badc49365f4dbf8dbae63a","unresolved":false,"context_lines":[{"line_number":41,"context_line":"    confirm_password \u003d forms.CharField("},{"line_number":42,"context_line":"        label\u003d_(\"Confirm Password\"),"},{"line_number":43,"context_line":"        widget\u003dforms.PasswordInput(render_value\u003dFalse),"},{"line_number":44,"context_line":"        required\u003dFalse)"},{"line_number":45,"context_line":"    type \u003d forms.ChoiceField(choices\u003d((\"\", \"\"),"},{"line_number":46,"context_line":"                                      (\"active_directory\", \"Active Directory\"),"},{"line_number":47,"context_line":"                                      (\"ldap\", \"LDAP\"),"}],"source_content_type":"text/x-python","patch_set":7,"id":"d0c75c74_359d9e2d","line":44,"range":{"start_line":44,"start_character":8,"end_line":44,"end_character":22},"in_reply_to":"3956bdc9_27e7ba74","updated":"2021-03-12 14:28:21.000000000","message":"I thought question is \"why field is not required\". I meant it is not \"required\u003dTrue\" because ValidationError is there. so no need to make confirm_password field required. Maybe I also misunderstood the question.","commit_id":"70a932372c58d1f9c80d3bbf747065a0ec324b67"},{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"2f287fd6659faeb5c930f9cb202e689bf1500c93","unresolved":false,"context_lines":[{"line_number":41,"context_line":"    confirm_password \u003d forms.CharField("},{"line_number":42,"context_line":"        label\u003d_(\"Confirm Password\"),"},{"line_number":43,"context_line":"        widget\u003dforms.PasswordInput(render_value\u003dFalse),"},{"line_number":44,"context_line":"        required\u003dFalse)"},{"line_number":45,"context_line":"    type \u003d forms.ChoiceField(choices\u003d((\"\", \"\"),"},{"line_number":46,"context_line":"                                      (\"active_directory\", \"Active Directory\"),"},{"line_number":47,"context_line":"                                      (\"ldap\", \"LDAP\"),"}],"source_content_type":"text/x-python","patch_set":7,"id":"3956bdc9_27e7ba74","line":44,"range":{"start_line":44,"start_character":8,"end_line":44,"end_character":22},"in_reply_to":"dc076bb2_bc8e4fa0","updated":"2021-03-12 14:17:47.000000000","message":"Well, that\u0027s the expected behavior, right? If a password is introduced and the confirm password field is empty then the form should raise the exception. Or maybe I understood this incorrectly.","commit_id":"70a932372c58d1f9c80d3bbf747065a0ec324b67"},{"author":{"_account_id":6413,"name":"Victoria Martinez de la Cruz","email":"victoria@redhat.com","username":"vkmc"},"change_message_id":"dd4a4556abed8f91a0520c44c8fe297d4ec27256","unresolved":true,"context_lines":[{"line_number":41,"context_line":"    confirm_password \u003d forms.CharField("},{"line_number":42,"context_line":"        label\u003d_(\"Confirm Password\"),"},{"line_number":43,"context_line":"        widget\u003dforms.PasswordInput(render_value\u003dFalse),"},{"line_number":44,"context_line":"        required\u003dFalse)"},{"line_number":45,"context_line":"    type \u003d forms.ChoiceField(choices\u003d((\"\", \"\"),"},{"line_number":46,"context_line":"                                      (\"active_directory\", \"Active Directory\"),"},{"line_number":47,"context_line":"                                      (\"ldap\", \"LDAP\"),"}],"source_content_type":"text/x-python","patch_set":7,"id":"f38e013a_4fc1d6b9","line":44,"range":{"start_line":44,"start_character":8,"end_line":44,"end_character":22},"in_reply_to":"e9877646_29cc8423","updated":"2021-03-12 00:21:50.000000000","message":"According to the docs these are all optional values. We also have these as optional in the CLI. Maybe we should make a smarter user input control?","commit_id":"70a932372c58d1f9c80d3bbf747065a0ec324b67"},{"author":{"_account_id":32531,"name":"Dina Saparbaeva","email":"dinas27@outlook.com","username":"disap"},"change_message_id":"21908372b52bd3fd053d0bd08ac6ef1fddc1c999","unresolved":false,"context_lines":[{"line_number":41,"context_line":"    confirm_password \u003d forms.CharField("},{"line_number":42,"context_line":"        label\u003d_(\"Confirm Password\"),"},{"line_number":43,"context_line":"        widget\u003dforms.PasswordInput(render_value\u003dFalse),"},{"line_number":44,"context_line":"        required\u003dFalse)"},{"line_number":45,"context_line":"    type \u003d forms.ChoiceField(choices\u003d((\"\", \"\"),"},{"line_number":46,"context_line":"                                      (\"active_directory\", \"Active Directory\"),"},{"line_number":47,"context_line":"                                      (\"ldap\", \"LDAP\"),"}],"source_content_type":"text/x-python","patch_set":7,"id":"dc076bb2_bc8e4fa0","line":44,"range":{"start_line":44,"start_character":8,"end_line":44,"end_character":22},"in_reply_to":"f38e013a_4fc1d6b9","updated":"2021-03-12 05:38:50.000000000","message":"I didn\u0027t make confirm_password required, because if user types password and then doesn\u0027t provide confirm_password, ValidationError \u0027Passwords do not match.\u0027 will be raised anyway as empty string won\u0027t match with whatever password user types.","commit_id":"70a932372c58d1f9c80d3bbf747065a0ec324b67"},{"author":{"_account_id":32531,"name":"Dina Saparbaeva","email":"dinas27@outlook.com","username":"disap"},"change_message_id":"21908372b52bd3fd053d0bd08ac6ef1fddc1c999","unresolved":false,"context_lines":[{"line_number":55,"context_line":"        data \u003d super(forms.Form, self).clean()"},{"line_number":56,"context_line":"        if \u0027password\u0027 in data:"},{"line_number":57,"context_line":"            if data[\u0027password\u0027] !\u003d data.get(\u0027confirm_password\u0027, None):"},{"line_number":58,"context_line":"                raise ValidationError(_(\u0027Passwords do not match.\u0027))"},{"line_number":59,"context_line":"        return data"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"    @sensitive_variables(\u0027data\u0027)"}],"source_content_type":"text/x-python","patch_set":7,"id":"9a9c2654_d66240e6","line":58,"updated":"2021-03-12 05:38:50.000000000","message":"Validation Error is here.","commit_id":"70a932372c58d1f9c80d3bbf747065a0ec324b67"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"1e3faf19ee96dbd66c59edf56fee6b787d5ada16","unresolved":true,"context_lines":[{"line_number":107,"context_line":"            raise ValidationError(_(\u0027Passwords do not match.\u0027))"},{"line_number":108,"context_line":"        return cleaned_data"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    @sensitive_variables(\u0027data\u0027)"},{"line_number":111,"context_line":"    def handle(self, request, data):"},{"line_number":112,"context_line":"        sec_service_id \u003d self.initial[\u0027sec_service_id\u0027]"},{"line_number":113,"context_line":"        try:"}],"source_content_type":"text/x-python","patch_set":7,"id":"e64166ea_c8800ab1","line":110,"range":{"start_line":110,"start_character":0,"end_line":110,"end_character":32},"updated":"2021-03-11 18:30:14.000000000","message":"very nice, Thank you!","commit_id":"70a932372c58d1f9c80d3bbf747065a0ec324b67"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"ae7b30efcb54f7fd95132451b9c401e62e601392","unresolved":true,"context_lines":[{"line_number":117,"context_line":"                ou\u003ddata[\u0027ou\u0027],"},{"line_number":118,"context_line":"                server\u003ddata[\u0027server\u0027],"},{"line_number":119,"context_line":"                domain\u003ddata[\u0027domain\u0027],"},{"line_number":120,"context_line":"                password\u003ddata[\u0027password\u0027],"},{"line_number":121,"context_line":"                user\u003ddata[\u0027user\u0027],"},{"line_number":122,"context_line":"                name\u003ddata[\u0027name\u0027],"},{"line_number":123,"context_line":"                description\u003ddata[\u0027description\u0027])"}],"source_content_type":"text/x-python","patch_set":7,"id":"e06f2ab5_15ad9bdb","line":120,"range":{"start_line":120,"start_character":16,"end_line":120,"end_character":42},"updated":"2021-03-12 08:15:30.000000000","message":"You must add this only if it isn\u0027t None; else we\u0027ll be overwriting the password each time.","commit_id":"70a932372c58d1f9c80d3bbf747065a0ec324b67"},{"author":{"_account_id":32531,"name":"Dina Saparbaeva","email":"dinas27@outlook.com","username":"disap"},"change_message_id":"98c9a58a232b2b47473ad377206b0e30aa2682c4","unresolved":false,"context_lines":[{"line_number":117,"context_line":"                ou\u003ddata[\u0027ou\u0027],"},{"line_number":118,"context_line":"                server\u003ddata[\u0027server\u0027],"},{"line_number":119,"context_line":"                domain\u003ddata[\u0027domain\u0027],"},{"line_number":120,"context_line":"                password\u003ddata[\u0027password\u0027],"},{"line_number":121,"context_line":"                user\u003ddata[\u0027user\u0027],"},{"line_number":122,"context_line":"                name\u003ddata[\u0027name\u0027],"},{"line_number":123,"context_line":"                description\u003ddata[\u0027description\u0027])"}],"source_content_type":"text/x-python","patch_set":7,"id":"a962580a_be27ce16","line":120,"range":{"start_line":120,"start_character":16,"end_line":120,"end_character":42},"in_reply_to":"e06f2ab5_15ad9bdb","updated":"2021-03-12 09:37:13.000000000","message":"Ack, thanks","commit_id":"70a932372c58d1f9c80d3bbf747065a0ec324b67"}],"manila_ui/dashboards/project/security_services/templates/security_services/_detail.html":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"1e3faf19ee96dbd66c59edf56fee6b787d5ada16","unresolved":true,"context_lines":[{"line_number":23,"context_line":"\u003cdiv class\u003d\"detail\"\u003e"},{"line_number":24,"context_line":"  \u003ch4\u003e{% trans \"Security Service Details\" %}\u003c/h4\u003e"},{"line_number":25,"context_line":"  \u003chr class\u003d\"header_rule\"\u003e"},{"line_number":26,"context_line":"  \u003cdl class\u003d\"dl-horizontal\"\u003e"},{"line_number":27,"context_line":"    \u003cdt\u003e{% trans \"DNS IP\" %}\u003c/dt\u003e"},{"line_number":28,"context_line":"    \u003cdd\u003e{{ sec_service.dns_ip }}\u003c/dd\u003e"},{"line_number":29,"context_line":"    \u003cdt\u003e{% trans \"Organizational Unit\" %}\u003c/dt\u003e"}],"source_content_type":"text/html","patch_set":7,"id":"03802b92_7833d43d","line":26,"range":{"start_line":26,"start_character":27,"end_line":26,"end_character":28},"updated":"2021-03-11 18:30:14.000000000","message":"Name, Security Service Type and Description would be appropriate here too.","commit_id":"70a932372c58d1f9c80d3bbf747065a0ec324b67"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"ae7b30efcb54f7fd95132451b9c401e62e601392","unresolved":true,"context_lines":[{"line_number":23,"context_line":"\u003cdiv class\u003d\"detail\"\u003e"},{"line_number":24,"context_line":"  \u003ch4\u003e{% trans \"Security Service Details\" %}\u003c/h4\u003e"},{"line_number":25,"context_line":"  \u003chr class\u003d\"header_rule\"\u003e"},{"line_number":26,"context_line":"  \u003cdl class\u003d\"dl-horizontal\"\u003e"},{"line_number":27,"context_line":"    \u003cdt\u003e{% trans \"DNS IP\" %}\u003c/dt\u003e"},{"line_number":28,"context_line":"    \u003cdd\u003e{{ sec_service.dns_ip }}\u003c/dd\u003e"},{"line_number":29,"context_line":"    \u003cdt\u003e{% trans \"Organizational Unit\" %}\u003c/dt\u003e"}],"source_content_type":"text/html","patch_set":7,"id":"000ae7f4_aa27cf48","line":26,"range":{"start_line":26,"start_character":27,"end_line":26,"end_character":28},"in_reply_to":"03802b92_7833d43d","updated":"2021-03-12 08:15:30.000000000","message":"ignore, they\u0027re in the overview above as you pointed out.","commit_id":"70a932372c58d1f9c80d3bbf747065a0ec324b67"}],"manila_ui/dashboards/project/security_services/views.py":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"1e3faf19ee96dbd66c59edf56fee6b787d5ada16","unresolved":true,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    def get_initial(self):"},{"line_number":81,"context_line":"        sec_service \u003d self.get_object()"},{"line_number":82,"context_line":"        return {\u0027sec_service_id\u0027: self.kwargs[\"sec_service_id\"],"},{"line_number":83,"context_line":"                \u0027dns_ip\u0027: sec_service.dns_ip,"},{"line_number":84,"context_line":"                \u0027ou\u0027: sec_service.ou,"},{"line_number":85,"context_line":"                \u0027server\u0027: sec_service.server,"},{"line_number":86,"context_line":"                \u0027domain\u0027: sec_service.domain,"},{"line_number":87,"context_line":"                \u0027user\u0027: sec_service.user,"},{"line_number":88,"context_line":"                \u0027name\u0027: sec_service.name,"},{"line_number":89,"context_line":"                \u0027description\u0027: sec_service.description"},{"line_number":90,"context_line":"                }"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"class CreateView(forms.ModalFormView):"}],"source_content_type":"text/x-python","patch_set":7,"id":"2753ecde_83a8985e","line":90,"range":{"start_line":82,"start_character":15,"end_line":90,"end_character":17},"updated":"2021-03-11 18:30:14.000000000","message":"nit: could improve the formatting here: https://docs.openstack.org/hacking/latest/user/hacking.html#dictionaries-lists","commit_id":"70a932372c58d1f9c80d3bbf747065a0ec324b67"},{"author":{"_account_id":32531,"name":"Dina Saparbaeva","email":"dinas27@outlook.com","username":"disap"},"change_message_id":"21908372b52bd3fd053d0bd08ac6ef1fddc1c999","unresolved":false,"context_lines":[{"line_number":79,"context_line":""},{"line_number":80,"context_line":"    def get_initial(self):"},{"line_number":81,"context_line":"        sec_service \u003d self.get_object()"},{"line_number":82,"context_line":"        return {\u0027sec_service_id\u0027: self.kwargs[\"sec_service_id\"],"},{"line_number":83,"context_line":"                \u0027dns_ip\u0027: sec_service.dns_ip,"},{"line_number":84,"context_line":"                \u0027ou\u0027: sec_service.ou,"},{"line_number":85,"context_line":"                \u0027server\u0027: sec_service.server,"},{"line_number":86,"context_line":"                \u0027domain\u0027: sec_service.domain,"},{"line_number":87,"context_line":"                \u0027user\u0027: sec_service.user,"},{"line_number":88,"context_line":"                \u0027name\u0027: sec_service.name,"},{"line_number":89,"context_line":"                \u0027description\u0027: sec_service.description"},{"line_number":90,"context_line":"                }"},{"line_number":91,"context_line":""},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"class CreateView(forms.ModalFormView):"}],"source_content_type":"text/x-python","patch_set":7,"id":"b7d51b79_cfac7e85","line":90,"range":{"start_line":82,"start_character":15,"end_line":90,"end_character":17},"in_reply_to":"2753ecde_83a8985e","updated":"2021-03-12 05:38:50.000000000","message":"Ack","commit_id":"70a932372c58d1f9c80d3bbf747065a0ec324b67"}]}