{"action":"created","issue":{"url":"https://api.github.com/repos/omu/nokul/issues/275","repository_url":"https://api.github.com/repos/omu/nokul","labels_url":"https://api.github.com/repos/omu/nokul/issues/275/labels{/name}","comments_url":"https://api.github.com/repos/omu/nokul/issues/275/comments","events_url":"https://api.github.com/repos/omu/nokul/issues/275/events","html_url":"https://github.com/omu/nokul/pull/275","id":358921938,"node_id":"MDExOlB1bGxSZXF1ZXN0MjE0NTM3OTQz","number":275,"title":"Configure administrative units","user":{"login":"ecmelkytz","id":937358,"node_id":"MDQ6VXNlcjkzNzM1OA==","avatar_url":"https://avatars2.githubusercontent.com/u/937358?v=4","gravatar_id":"","url":"https://api.github.com/users/ecmelkytz","html_url":"https://github.com/ecmelkytz","followers_url":"https://api.github.com/users/ecmelkytz/followers","following_url":"https://api.github.com/users/ecmelkytz/following{/other_user}","gists_url":"https://api.github.com/users/ecmelkytz/gists{/gist_id}","starred_url":"https://api.github.com/users/ecmelkytz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ecmelkytz/subscriptions","organizations_url":"https://api.github.com/users/ecmelkytz/orgs","repos_url":"https://api.github.com/users/ecmelkytz/repos","events_url":"https://api.github.com/users/ecmelkytz/events{/privacy}","received_events_url":"https://api.github.com/users/ecmelkytz/received_events","type":"User","site_admin":false},"labels":[{"id":984146674,"node_id":"MDU6TGFiZWw5ODQxNDY2NzQ=","url":"https://api.github.com/repos/omu/nokul/labels/feature-step","name":"feature-step","color":"e59290","default":false}],"state":"open","locked":false,"assignee":{"login":"ecmelkytz","id":937358,"node_id":"MDQ6VXNlcjkzNzM1OA==","avatar_url":"https://avatars2.githubusercontent.com/u/937358?v=4","gravatar_id":"","url":"https://api.github.com/users/ecmelkytz","html_url":"https://github.com/ecmelkytz","followers_url":"https://api.github.com/users/ecmelkytz/followers","following_url":"https://api.github.com/users/ecmelkytz/following{/other_user}","gists_url":"https://api.github.com/users/ecmelkytz/gists{/gist_id}","starred_url":"https://api.github.com/users/ecmelkytz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ecmelkytz/subscriptions","organizations_url":"https://api.github.com/users/ecmelkytz/orgs","repos_url":"https://api.github.com/users/ecmelkytz/repos","events_url":"https://api.github.com/users/ecmelkytz/events{/privacy}","received_events_url":"https://api.github.com/users/ecmelkytz/received_events","type":"User","site_admin":false},"assignees":[{"login":"ecmelkytz","id":937358,"node_id":"MDQ6VXNlcjkzNzM1OA==","avatar_url":"https://avatars2.githubusercontent.com/u/937358?v=4","gravatar_id":"","url":"https://api.github.com/users/ecmelkytz","html_url":"https://github.com/ecmelkytz","followers_url":"https://api.github.com/users/ecmelkytz/followers","following_url":"https://api.github.com/users/ecmelkytz/following{/other_user}","gists_url":"https://api.github.com/users/ecmelkytz/gists{/gist_id}","starred_url":"https://api.github.com/users/ecmelkytz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ecmelkytz/subscriptions","organizations_url":"https://api.github.com/users/ecmelkytz/orgs","repos_url":"https://api.github.com/users/ecmelkytz/repos","events_url":"https://api.github.com/users/ecmelkytz/events{/privacy}","received_events_url":"https://api.github.com/users/ecmelkytz/received_events","type":"User","site_admin":false}],"milestone":null,"comments":2,"created_at":"2018-09-11T07:31:35Z","updated_at":"2018-09-11T12:47:04Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/omu/nokul/pulls/275","html_url":"https://github.com/omu/nokul/pull/275","diff_url":"https://github.com/omu/nokul/pull/275.diff","patch_url":"https://github.com/omu/nokul/pull/275.patch"},"body":"**Bu PR'in yaptığı işi/değişikliği ve bu işi/değişikliği neden yaptığını açıklayın:**\r\nİdari birimlerin sisteme entegre edilmesini sağlar.\r\n\r\n[//]: # (Kısa ve net bir şekilde bu PR'e neden ihtiyaç var, ne iş/değişiklik yapıyor açıklanmalıdır.)\r\n\r\n**İlgili iş kayıtları:**\r\n#236, #260 \r\n\r\n[//]: # (Bu PR ile ilişkili iş kayıtlarının ID numaraları, örn. #123456, #123457, #123458, listeleyin.)\r\n\r\n**Kapatılacak iş kayıtları:**\r\n#236, #260 \r\n\r\n[//]: # (Bu PR merge edildiğinde hangi iş kayıtları kapatılacak, `Closes`, `Fixes` gibi anahtar kelimeler ile birlikte ID numaralarını listeleyin.)\r\n\r\n**Veritabanına etkileri:**\r\nVeritabanı drop edilmesin diye `Unit` modeli için detsis_id ve yoksis_id kısıtlamalarını değiştirecek bir migration dosyası eklendi.\r\n\r\nİdari birimleri oluşturmak için çalıştırılacak komut:  `rails kaysis:import_administrative_units`\r\n\r\n[//]: # (Bu PR merge edildiğinde veritabanı üzerinde herhangi bir değişiklik -migration, seed, add/drop vb.- yapılması gerekecek mi açıklayın.)\r\n\r\n**Sistem/Ops etkileri:**\r\n\r\n[//]: # (Bu PR merge edildiğinde sunucular üzerinde bir değişiklik - paket kurulması, buildpack eklenmesi vb. - yapılması gerekecekmi, açıklayın.)\r\n\r\n**Kontrol listesi:**\r\n\r\n* [x] Açtığınız PR'in başlığı kurallara (sadece ilk harf büyük, emir kipinde problem cümlesi vb.) uygun mu?\r\n* [x] [Katkı sağlama dokümanını](https://github.com/omu/nokul/CONTRIBUTING.md) okudunuz mu?\r\n* [ ] Yaptığınız iş/değişikliği dokümante ettiniz mi?\r\n* [x] Yaptığınız iş/değişikliğin testlerini yazdınız mı?\r\n* [x] Test coverage oranını kontrol ettiniz mi?\r\n* [x] Kod kalitesi (karma) ve test suite dahil olmak üzere tüm entegre kontroller başarıyla geçiyor mu?\r\n* [x] Kendinizi bu PR'e assign ettiniz mi?\r\n* [x] Yaptığınız iş/değişiklik ile ilgili **tüm** proje üyelerinden review talep ettiniz mi?\r\n* [x] Gerekli etiketleri (bug, feature-step, help-wanted vb.) girdiniz mi?\r\n\r\n**Ek içerik:**\r\n\r\n[//]: # (Kaynaklar, dış bağlantılar, ekran görüntüleri, örnek çözümler ve benzeri diğer kaynakları ekleyiniz.)\r\n"},"comment":{"url":"https://api.github.com/repos/omu/nokul/issues/comments/420261548","html_url":"https://github.com/omu/nokul/pull/275#issuecomment-420261548","issue_url":"https://api.github.com/repos/omu/nokul/issues/275","id":420261548,"node_id":"MDEyOklzc3VlQ29tbWVudDQyMDI2MTU0OA==","user":{"login":"roktas","id":47842,"node_id":"MDQ6VXNlcjQ3ODQy","avatar_url":"https://avatars2.githubusercontent.com/u/47842?v=4","gravatar_id":"","url":"https://api.github.com/users/roktas","html_url":"https://github.com/roktas","followers_url":"https://api.github.com/users/roktas/followers","following_url":"https://api.github.com/users/roktas/following{/other_user}","gists_url":"https://api.github.com/users/roktas/gists{/gist_id}","starred_url":"https://api.github.com/users/roktas/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/roktas/subscriptions","organizations_url":"https://api.github.com/users/roktas/orgs","repos_url":"https://api.github.com/users/roktas/repos","events_url":"https://api.github.com/users/roktas/events{/privacy}","received_events_url":"https://api.github.com/users/roktas/received_events","type":"User","site_admin":false},"created_at":"2018-09-11T12:47:04Z","updated_at":"2018-09-11T12:47:04Z","author_association":"MEMBER","body":"@isubas review'e review :-)  **Never ever use absolute paths in code**\r\n\r\n```ruby\r\npath = '/home/isubas/project/nokul/db/static_data/test.yml'\r\n```\r\nyerine (bir kabul yaparak)\r\n\r\n```ruby\r\npath = 'db/static_data/test.yml'\r\n```\r\n\r\nP.S. Eliniz hiç bir zaman alışmasın diye not ediyorum."}}
{"actor":{"display_login":"roktas"}}