{"action":"created","issue":{"url":"https://api.github.com/repos/Geonovum/technisch-register/issues/33","labels_url":"https://api.github.com/repos/Geonovum/technisch-register/issues/33/labels{/name}","comments_url":"https://api.github.com/repos/Geonovum/technisch-register/issues/33/comments","events_url":"https://api.github.com/repos/Geonovum/technisch-register/issues/33/events","html_url":"https://github.com/Geonovum/technisch-register/pull/33","id":122309370,"number":33,"title":"Modellen geclusterd","user":{"login":"nicojanbakker","id":15192014,"avatar_url":"https://avatars.githubusercontent.com/u/15192014?v=3","gravatar_id":"","url":"https://api.github.com/users/nicojanbakker","html_url":"https://github.com/nicojanbakker","followers_url":"https://api.github.com/users/nicojanbakker/followers","following_url":"https://api.github.com/users/nicojanbakker/following{/other_user}","gists_url":"https://api.github.com/users/nicojanbakker/gists{/gist_id}","starred_url":"https://api.github.com/users/nicojanbakker/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nicojanbakker/subscriptions","organizations_url":"https://api.github.com/users/nicojanbakker/orgs","repos_url":"https://api.github.com/users/nicojanbakker/repos","events_url":"https://api.github.com/users/nicojanbakker/events{/privacy}","received_events_url":"https://api.github.com/users/nicojanbakker/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2015-12-15T16:21:06Z","updated_at":"2015-12-18T13:33:40Z","closed_at":"2015-12-16T14:14:55Z","pull_request":{"url":"https://api.github.com/repos/Geonovum/technisch-register/pulls/33","html_url":"https://github.com/Geonovum/technisch-register/pull/33","diff_url":"https://github.com/Geonovum/technisch-register/pull/33.diff","patch_url":"https://github.com/Geonovum/technisch-register/pull/33.patch"},"body":"Mogelijkheid tot het clusteren van modellen toegevoegd. Clusters staan beschreven in cluster.json. In repos.json is per model een veldje 'cluster' toegevoegd.\r\n"},"comment":{"url":"https://api.github.com/repos/Geonovum/technisch-register/issues/comments/165779663","html_url":"https://github.com/Geonovum/technisch-register/pull/33#issuecomment-165779663","issue_url":"https://api.github.com/repos/Geonovum/technisch-register/issues/33","id":165779663,"user":{"login":"nicojanbakker","id":15192014,"avatar_url":"https://avatars.githubusercontent.com/u/15192014?v=3","gravatar_id":"","url":"https://api.github.com/users/nicojanbakker","html_url":"https://github.com/nicojanbakker","followers_url":"https://api.github.com/users/nicojanbakker/followers","following_url":"https://api.github.com/users/nicojanbakker/following{/other_user}","gists_url":"https://api.github.com/users/nicojanbakker/gists{/gist_id}","starred_url":"https://api.github.com/users/nicojanbakker/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nicojanbakker/subscriptions","organizations_url":"https://api.github.com/users/nicojanbakker/orgs","repos_url":"https://api.github.com/users/nicojanbakker/repos","events_url":"https://api.github.com/users/nicojanbakker/events{/privacy}","received_events_url":"https://api.github.com/users/nicojanbakker/received_events","type":"User","site_admin":false},"created_at":"2015-12-18T13:33:39Z","updated_at":"2015-12-18T13:33:39Z","body":"Ah oke, op die manier. Daar had ik even overheen gelezen. os.symlink is nu\n'uitgezet'. Toen ik de code vervolgens ging runnen herkende hij\nclusters.json niet. cluster_path verwees namelijk naar clusters.json\nterwijl het bestand cluster.json heet. Dus ik heb de bestandsnaam een s'je\nextra gegeven. Vervolgens moest ik zelf de map staging en repos toevoegen\nom verder te komen. Maar nu runt de code wel goed. Ook meerdere keren\nachter elkaar runnen gaat dan goed.\n\nWat ik heb gedaan om de code werkend te krijgen (evt ter aanvulling op\nREADME file):\n1- De zaken beschreven in de README\n2- os.symlink uitzetten met #\n3- handmatig: /repos/imkl toevoegen aan ....../technisch-register\n4- handmatig: /staging toevoegen aan ..../technisch-register (evt /register\ntoevoegen om die ook te kunnen testen)\n5- Bij committen os.symlink weer aan zetten\n\n\n\n\nOp 18 december 2015 12:36 schreef Simeon Nedkov <notifications@github.com>:\n\n> Als Python iets niet kan importeren betekent het vaak dat je een module\n> mist. In dit geval komt het echter, denk ik, doordat Windows het fenomeen\n> symlink niet kent (daar heet het een junction point) en os.symlink\n> waarschijnlijk niet geimplementeerd is. os.symlink wordt niet in code\n> gebruikt dus je kan het weghalen.\n>\n> Re: settings.py het is de bedoeling dat je settings-example.py hernoemt\n> naar settings.py. Zie laatste regel van installing in de README:\n>\n> [image: screen shot 2015-12-18 at 12 26 02]\n> <https://cloud.githubusercontent.com/assets/778666/11895914/9167e8e6-a582-11e5-822e-705214643e20.png>\n>\n> settings.py is niet in de repo opgenomen omdat het 1) per deployment\n> locatie kan verschillen (i.e. de server heeft vaak andere configuratie\n> parameters dan lokale dev machines) en 2) daar vaak wachtwoorden in\n> opgeslagen worden (voor bijv. databases, APIs, etc.) die je niet in een\n> publieke repo zichtbaar wilt maken. Je levert daarom een dummy settings\n> bestand met placeholders voor de verschillende variabelen. Zie mijn eerdere\n> uitleg:\n>\n> [image: screen shot 2015-12-18 at 12 20 15]\n> <https://cloud.githubusercontent.com/assets/778666/11896048/a819e584-a583-11e5-9eaf-f3a52cbe38a1.png>\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/Geonovum/technisch-register/pull/33#issuecomment-165754407>\n> .\n>\n"}}