{"action":"created","issue":{"url":"https://api.github.com/repos/cordery/django-languages-plus/issues/1","labels_url":"https://api.github.com/repos/cordery/django-languages-plus/issues/1/labels{/name}","comments_url":"https://api.github.com/repos/cordery/django-languages-plus/issues/1/comments","events_url":"https://api.github.com/repos/cordery/django-languages-plus/issues/1/events","html_url":"https://github.com/cordery/django-languages-plus/pull/1","id":53957635,"number":1,"title":"Support for Python 3","user":{"login":"luiscberrocal","id":976869,"avatar_url":"https://avatars.githubusercontent.com/u/976869?v=3","gravatar_id":"","url":"https://api.github.com/users/luiscberrocal","html_url":"https://github.com/luiscberrocal","followers_url":"https://api.github.com/users/luiscberrocal/followers","following_url":"https://api.github.com/users/luiscberrocal/following{/other_user}","gists_url":"https://api.github.com/users/luiscberrocal/gists{/gist_id}","starred_url":"https://api.github.com/users/luiscberrocal/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/luiscberrocal/subscriptions","organizations_url":"https://api.github.com/users/luiscberrocal/orgs","repos_url":"https://api.github.com/users/luiscberrocal/repos","events_url":"https://api.github.com/users/luiscberrocal/events{/privacy}","received_events_url":"https://api.github.com/users/luiscberrocal/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-01-10T13:03:13Z","updated_at":"2015-01-10T13:30:48Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cordery/django-languages-plus/pulls/1","html_url":"https://github.com/cordery/django-languages-plus/pull/1","diff_url":"https://github.com/cordery/django-languages-plus/pull/1.diff","patch_url":"https://github.com/cordery/django-languages-plus/pull/1.patch"},"body":"I really like your app. When I added support for Python 3 the tests started breaking in order to test I added countries plus as a dependency and changed the way the fixtures load. I seems with autoload the tests would crash because the could not find the countries model. I changed it so the fixtures would load with the migrations. \r\n\r\nThis will stop autoloading fixtures on previous versions on django (<1.7). I'm willing to write a command to load the fixtures y you would be kind enough to accept my pull request.\r\n\r\nThanks!!"},"comment":{"url":"https://api.github.com/repos/cordery/django-languages-plus/issues/comments/69455768","html_url":"https://github.com/cordery/django-languages-plus/pull/1#issuecomment-69455768","issue_url":"https://api.github.com/repos/cordery/django-languages-plus/issues/1","id":69455768,"user":{"login":"cordery","id":274325,"avatar_url":"https://avatars.githubusercontent.com/u/274325?v=3","gravatar_id":"","url":"https://api.github.com/users/cordery","html_url":"https://github.com/cordery","followers_url":"https://api.github.com/users/cordery/followers","following_url":"https://api.github.com/users/cordery/following{/other_user}","gists_url":"https://api.github.com/users/cordery/gists{/gist_id}","starred_url":"https://api.github.com/users/cordery/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cordery/subscriptions","organizations_url":"https://api.github.com/users/cordery/orgs","repos_url":"https://api.github.com/users/cordery/repos","events_url":"https://api.github.com/users/cordery/events{/privacy}","received_events_url":"https://api.github.com/users/cordery/received_events","type":"User","site_admin":false},"created_at":"2015-01-10T13:30:48Z","updated_at":"2015-01-10T13:30:48Z","body":"Hi Luis,\r\n\r\nI'm glad you are finding it useful!  Thank you for the PR.  I appreciate the code cleanup and python 3 compatibility changes.  Regarding the changes to fixtures, before I accept your PR, I have in my head that a few months ago when I looked at this I discovered that there was a bug in Django 1.7 > 1.7.1 that caused the fixtures to be loaded in unit tests, see https://code.djangoproject.com/ticket/23699   I was waiting for this bug to be resolved before I figured out how to handle fixtures to be compatible with both Django 1.7, and earlier versions.  I believe my approach was that they should be loaded by a migration in Django 1.7, and loaded automatically in earlier versions (so they would still need to be called initial_data.json).\r\n\r\nThis bug has been fixed in 1.7.2 (https://docs.djangoproject.com/en/1.7/releases/1.7.2/), released last week.   \r\n\r\nAs I don't have any of this loaded into my current context, If you wouldn't mind taking a moment to re-evaluate your changes in the light of the Django 1.7.2 change to the way fixtures are handled in unit tests, and see if your changes are still the best way to handle fixtures, I would appreciate it.\r\n\r\nAs a second note, I had originally intended for the two packages (countries-plus and languages-plus) to not have inter dependencies, after all, if they do they should just be merged.  Is it possible to achieve the desired outcome without setting an explicit dependency?  \r\n\r\nThank you,\r\n\r\nAndrew"}}