{"action":"created","issue":{"url":"https://api.github.com/repos/Artory/drf-hal-json/issues/25","repository_url":"https://api.github.com/repos/Artory/drf-hal-json","labels_url":"https://api.github.com/repos/Artory/drf-hal-json/issues/25/labels{/name}","comments_url":"https://api.github.com/repos/Artory/drf-hal-json/issues/25/comments","events_url":"https://api.github.com/repos/Artory/drf-hal-json/issues/25/events","html_url":"https://github.com/Artory/drf-hal-json/issues/25","id":249513790,"number":25,"title":"SlugRelatedField and _links","user":{"login":"claytondaley","id":1097439,"avatar_url":"https://avatars1.githubusercontent.com/u/1097439?v=4","gravatar_id":"","url":"https://api.github.com/users/claytondaley","html_url":"https://github.com/claytondaley","followers_url":"https://api.github.com/users/claytondaley/followers","following_url":"https://api.github.com/users/claytondaley/following{/other_user}","gists_url":"https://api.github.com/users/claytondaley/gists{/gist_id}","starred_url":"https://api.github.com/users/claytondaley/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/claytondaley/subscriptions","organizations_url":"https://api.github.com/users/claytondaley/orgs","repos_url":"https://api.github.com/users/claytondaley/repos","events_url":"https://api.github.com/users/claytondaley/events{/privacy}","received_events_url":"https://api.github.com/users/claytondaley/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-08-11T00:41:19Z","updated_at":"2017-08-11T01:00:18Z","closed_at":null,"body":"`SlugRelatedField` is getting put into the `_links` section even if it's not a URI (let alone a valid one).\r\n\r\nI'm using the `SlugRelatedField` to extract the name of a Django group.  I need the group name to tell a user who's responsible for a task (if that user is not).  The groups are not available over REST so it doesn't make sense to represent them as a linkable resource.  As far as REST is concerned, the slug is just an attribute of the object.\r\n\r\nWhile `SlugRelatedField` could be used to pick a URI off an object, this behavior is not the appropriate default.  Instead, I believe this feature should require a dedicated class asserting that it's representing a `UriSlugRelatedField`."},"comment":{"url":"https://api.github.com/repos/Artory/drf-hal-json/issues/comments/321711310","html_url":"https://github.com/Artory/drf-hal-json/issues/25#issuecomment-321711310","issue_url":"https://api.github.com/repos/Artory/drf-hal-json/issues/25","id":321711310,"user":{"login":"claytondaley","id":1097439,"avatar_url":"https://avatars1.githubusercontent.com/u/1097439?v=4","gravatar_id":"","url":"https://api.github.com/users/claytondaley","html_url":"https://github.com/claytondaley","followers_url":"https://api.github.com/users/claytondaley/followers","following_url":"https://api.github.com/users/claytondaley/following{/other_user}","gists_url":"https://api.github.com/users/claytondaley/gists{/gist_id}","starred_url":"https://api.github.com/users/claytondaley/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/claytondaley/subscriptions","organizations_url":"https://api.github.com/users/claytondaley/orgs","repos_url":"https://api.github.com/users/claytondaley/repos","events_url":"https://api.github.com/users/claytondaley/events{/privacy}","received_events_url":"https://api.github.com/users/claytondaley/received_events","type":"User","site_admin":false},"created_at":"2017-08-11T01:00:18Z","updated_at":"2017-08-11T01:00:18Z","body":"If you agree in principle, I need some guidance before I can submit a PR.  I'm not clear which fields can/should be completely removed from the check, which fields (e.g. `HalHyperlinkedRelatedField` and `HalHyperlinkedIdentityField`) need to be included once their ancestors are removed, and which need new `Hal`-derived variants.\r\n\r\nI suspect this change won't need new tests because the existing tests for non-`Hal`-derived objects can simply be converted to cover the new `Hal`-derived ones."}}
{"actor":{"display_login":"claytondaley"}}