{"action":"created","comment":{"url":"https://api.github.com/repos/json-api/json-api/pulls/comments/37527159","id":37527159,"diff_hunk":"@@ -1095,6 +1095,10 @@ strategy might use query parameters such as `page[number]` and `page[size]`,\n an offset-based strategy might use `page[offset]` and `page[limit]`, while a\n cursor-based strategy might use `page[cursor]`.\n \n+> Note: The above example URI shows unencoded `[` and `]` characters simply","path":"format/index.md","position":4,"original_position":4,"commit_id":"421eff8d1f492b1aac9e6dfb8ae9d5db99f71759","original_commit_id":"421eff8d1f492b1aac9e6dfb8ae9d5db99f71759","user":{"login":"ethanresnick","id":471894,"avatar_url":"https://avatars.githubusercontent.com/u/471894?v=3","gravatar_id":"","url":"https://api.github.com/users/ethanresnick","html_url":"https://github.com/ethanresnick","followers_url":"https://api.github.com/users/ethanresnick/followers","following_url":"https://api.github.com/users/ethanresnick/following{/other_user}","gists_url":"https://api.github.com/users/ethanresnick/gists{/gist_id}","starred_url":"https://api.github.com/users/ethanresnick/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ethanresnick/subscriptions","organizations_url":"https://api.github.com/users/ethanresnick/orgs","repos_url":"https://api.github.com/users/ethanresnick/repos","events_url":"https://api.github.com/users/ethanresnick/events{/privacy}","received_events_url":"https://api.github.com/users/ethanresnick/received_events","type":"User","site_admin":false},"body":"Thanks again for this!\r\n\r\nTwo small things: since we don't actually have a URI here (but query parameters), I think this should say \"The above example query parameters use unencoded `[` and `]` characters simply for readability.\" Then, since this note is already in the base spec, it should instead end: \"In practice, these characters must be percent-encoded, per the requirements in [RFC 3986](http://tools.ietf.org/html/rfc3986#section-3.4).\"","created_at":"2015-08-20T13:17:54Z","updated_at":"2015-08-20T13:17:54Z","html_url":"https://github.com/json-api/json-api/pull/849#discussion_r37527159","pull_request_url":"https://api.github.com/repos/json-api/json-api/pulls/849","_links":{"self":{"href":"https://api.github.com/repos/json-api/json-api/pulls/comments/37527159"},"html":{"href":"https://github.com/json-api/json-api/pull/849#discussion_r37527159"},"pull_request":{"href":"https://api.github.com/repos/json-api/json-api/pulls/849"}}},"pull_request":{"url":"https://api.github.com/repos/json-api/json-api/pulls/849","id":42927467,"html_url":"https://github.com/json-api/json-api/pull/849","diff_url":"https://github.com/json-api/json-api/pull/849.diff","patch_url":"https://github.com/json-api/json-api/pull/849.patch","issue_url":"https://api.github.com/repos/json-api/json-api/issues/849","number":849,"state":"open","locked":false,"title":"Update index.md","user":{"login":"afilina","id":199835,"avatar_url":"https://avatars.githubusercontent.com/u/199835?v=3","gravatar_id":"","url":"https://api.github.com/users/afilina","html_url":"https://github.com/afilina","followers_url":"https://api.github.com/users/afilina/followers","following_url":"https://api.github.com/users/afilina/following{/other_user}","gists_url":"https://api.github.com/users/afilina/gists{/gist_id}","starred_url":"https://api.github.com/users/afilina/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/afilina/subscriptions","organizations_url":"https://api.github.com/users/afilina/orgs","repos_url":"https://api.github.com/users/afilina/repos","events_url":"https://api.github.com/users/afilina/events{/privacy}","received_events_url":"https://api.github.com/users/afilina/received_events","type":"User","site_admin":false},"body":"Explicit encoding mention for pagination example.","created_at":"2015-08-20T13:13:13Z","updated_at":"2015-08-20T13:17:54Z","closed_at":null,"merged_at":null,"merge_commit_sha":"600168140a9fd2d7386f53f7992cf8fb30eaa14f","assignee":null,"milestone":null,"commits_url":"https://api.github.com/repos/json-api/json-api/pulls/849/commits","review_comments_url":"https://api.github.com/repos/json-api/json-api/pulls/849/comments","review_comment_url":"https://api.github.com/repos/json-api/json-api/pulls/comments{/number}","comments_url":"https://api.github.com/repos/json-api/json-api/issues/849/comments","statuses_url":"https://api.github.com/repos/json-api/json-api/statuses/421eff8d1f492b1aac9e6dfb8ae9d5db99f71759","head":{"label":"afilina:patch-1","ref":"patch-1","sha":"421eff8d1f492b1aac9e6dfb8ae9d5db99f71759","user":{"login":"afilina","id":199835,"avatar_url":"https://avatars.githubusercontent.com/u/199835?v=3","gravatar_id":"","url":"https://api.github.com/users/afilina","html_url":"https://github.com/afilina","followers_url":"https://api.github.com/users/afilina/followers","following_url":"https://api.github.com/users/afilina/following{/other_user}","gists_url":"https://api.github.com/users/afilina/gists{/gist_id}","starred_url":"https://api.github.com/users/afilina/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/afilina/subscriptions","organizations_url":"https://api.github.com/users/afilina/orgs","repos_url":"https://api.github.com/users/afilina/repos","events_url":"https://api.github.com/users/afilina/events{/privacy}","received_events_url":"https://api.github.com/users/afilina/received_events","type":"User","site_admin":false},"repo":{"id":41096220,"name":"json-api","full_name":"afilina/json-api","owner":{"login":"afilina","id":199835,"avatar_url":"https://avatars.githubusercontent.com/u/199835?v=3","gravatar_id":"","url":"https://api.github.com/users/afilina","html_url":"https://github.com/afilina","followers_url":"https://api.github.com/users/afilina/followers","following_url":"https://api.github.com/users/afilina/following{/other_user}","gists_url":"https://api.github.com/users/afilina/gists{/gist_id}","starred_url":"https://api.github.com/users/afilina/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/afilina/subscriptions","organizations_url":"https://api.github.com/users/afilina/orgs","repos_url":"https://api.github.com/users/afilina/repos","events_url":"https://api.github.com/users/afilina/events{/privacy}","received_events_url":"https://api.github.com/users/afilina/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/afilina/json-api","description":"A specification for building JSON APIs","fork":true,"url":"https://api.github.com/repos/afilina/json-api","forks_url":"https://api.github.com/repos/afilina/json-api/forks","keys_url":"https://api.github.com/repos/afilina/json-api/keys{/key_id}","collaborators_url":"https://api.github.com/repos/afilina/json-api/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/afilina/json-api/teams","hooks_url":"https://api.github.com/repos/afilina/json-api/hooks","issue_events_url":"https://api.github.com/repos/afilina/json-api/issues/events{/number}","events_url":"https://api.github.com/repos/afilina/json-api/events","assignees_url":"https://api.github.com/repos/afilina/json-api/assignees{/user}","branches_url":"https://api.github.com/repos/afilina/json-api/branches{/branch}","tags_url":"https://api.github.com/repos/afilina/json-api/tags","blobs_url":"https://api.github.com/repos/afilina/json-api/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/afilina/json-api/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/afilina/json-api/git/refs{/sha}","trees_url":"https://api.github.com/repos/afilina/json-api/git/trees{/sha}","statuses_url":"https://api.github.com/repos/afilina/json-api/statuses/{sha}","languages_url":"https://api.github.com/repos/afilina/json-api/languages","stargazers_url":"https://api.github.com/repos/afilina/json-api/stargazers","contributors_url":"https://api.github.com/repos/afilina/json-api/contributors","subscribers_url":"https://api.github.com/repos/afilina/json-api/subscribers","subscription_url":"https://api.github.com/repos/afilina/json-api/subscription","commits_url":"https://api.github.com/repos/afilina/json-api/commits{/sha}","git_commits_url":"https://api.github.com/repos/afilina/json-api/git/commits{/sha}","comments_url":"https://api.github.com/repos/afilina/json-api/comments{/number}","issue_comment_url":"https://api.github.com/repos/afilina/json-api/issues/comments{/number}","contents_url":"https://api.github.com/repos/afilina/json-api/contents/{+path}","compare_url":"https://api.github.com/repos/afilina/json-api/compare/{base}...{head}","merges_url":"https://api.github.com/repos/afilina/json-api/merges","archive_url":"https://api.github.com/repos/afilina/json-api/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/afilina/json-api/downloads","issues_url":"https://api.github.com/repos/afilina/json-api/issues{/number}","pulls_url":"https://api.github.com/repos/afilina/json-api/pulls{/number}","milestones_url":"https://api.github.com/repos/afilina/json-api/milestones{/number}","notifications_url":"https://api.github.com/repos/afilina/json-api/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/afilina/json-api/labels{/name}","releases_url":"https://api.github.com/repos/afilina/json-api/releases{/id}","created_at":"2015-08-20T13:09:04Z","updated_at":"2015-08-20T13:09:05Z","pushed_at":"2015-08-20T13:13:08Z","git_url":"git://github.com/afilina/json-api.git","ssh_url":"git@github.com:afilina/json-api.git","clone_url":"https://github.com/afilina/json-api.git","svn_url":"https://github.com/afilina/json-api","homepage":"http://jsonapi.org","size":4700,"stargazers_count":0,"watchers_count":0,"language":"CSS","has_issues":false,"has_downloads":true,"has_wiki":false,"has_pages":true,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"gh-pages"}},"base":{"label":"json-api:contributing","ref":"contributing","sha":"7edf40c3c41724aecc658f08a2e739125502a0b0","user":{"login":"json-api","id":4341694,"avatar_url":"https://avatars.githubusercontent.com/u/4341694?v=3","gravatar_id":"","url":"https://api.github.com/users/json-api","html_url":"https://github.com/json-api","followers_url":"https://api.github.com/users/json-api/followers","following_url":"https://api.github.com/users/json-api/following{/other_user}","gists_url":"https://api.github.com/users/json-api/gists{/gist_id}","starred_url":"https://api.github.com/users/json-api/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/json-api/subscriptions","organizations_url":"https://api.github.com/users/json-api/orgs","repos_url":"https://api.github.com/users/json-api/repos","events_url":"https://api.github.com/users/json-api/events{/privacy}","received_events_url":"https://api.github.com/users/json-api/received_events","type":"Organization","site_admin":false},"repo":{"id":9859154,"name":"json-api","full_name":"json-api/json-api","owner":{"login":"json-api","id":4341694,"avatar_url":"https://avatars.githubusercontent.com/u/4341694?v=3","gravatar_id":"","url":"https://api.github.com/users/json-api","html_url":"https://github.com/json-api","followers_url":"https://api.github.com/users/json-api/followers","following_url":"https://api.github.com/users/json-api/following{/other_user}","gists_url":"https://api.github.com/users/json-api/gists{/gist_id}","starred_url":"https://api.github.com/users/json-api/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/json-api/subscriptions","organizations_url":"https://api.github.com/users/json-api/orgs","repos_url":"https://api.github.com/users/json-api/repos","events_url":"https://api.github.com/users/json-api/events{/privacy}","received_events_url":"https://api.github.com/users/json-api/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/json-api/json-api","description":"A specification for building JSON APIs","fork":false,"url":"https://api.github.com/repos/json-api/json-api","forks_url":"https://api.github.com/repos/json-api/json-api/forks","keys_url":"https://api.github.com/repos/json-api/json-api/keys{/key_id}","collaborators_url":"https://api.github.com/repos/json-api/json-api/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/json-api/json-api/teams","hooks_url":"https://api.github.com/repos/json-api/json-api/hooks","issue_events_url":"https://api.github.com/repos/json-api/json-api/issues/events{/number}","events_url":"https://api.github.com/repos/json-api/json-api/events","assignees_url":"https://api.github.com/repos/json-api/json-api/assignees{/user}","branches_url":"https://api.github.com/repos/json-api/json-api/branches{/branch}","tags_url":"https://api.github.com/repos/json-api/json-api/tags","blobs_url":"https://api.github.com/repos/json-api/json-api/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/json-api/json-api/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/json-api/json-api/git/refs{/sha}","trees_url":"https://api.github.com/repos/json-api/json-api/git/trees{/sha}","statuses_url":"https://api.github.com/repos/json-api/json-api/statuses/{sha}","languages_url":"https://api.github.com/repos/json-api/json-api/languages","stargazers_url":"https://api.github.com/repos/json-api/json-api/stargazers","contributors_url":"https://api.github.com/repos/json-api/json-api/contributors","subscribers_url":"https://api.github.com/repos/json-api/json-api/subscribers","subscription_url":"https://api.github.com/repos/json-api/json-api/subscription","commits_url":"https://api.github.com/repos/json-api/json-api/commits{/sha}","git_commits_url":"https://api.github.com/repos/json-api/json-api/git/commits{/sha}","comments_url":"https://api.github.com/repos/json-api/json-api/comments{/number}","issue_comment_url":"https://api.github.com/repos/json-api/json-api/issues/comments{/number}","contents_url":"https://api.github.com/repos/json-api/json-api/contents/{+path}","compare_url":"https://api.github.com/repos/json-api/json-api/compare/{base}...{head}","merges_url":"https://api.github.com/repos/json-api/json-api/merges","archive_url":"https://api.github.com/repos/json-api/json-api/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/json-api/json-api/downloads","issues_url":"https://api.github.com/repos/json-api/json-api/issues{/number}","pulls_url":"https://api.github.com/repos/json-api/json-api/pulls{/number}","milestones_url":"https://api.github.com/repos/json-api/json-api/milestones{/number}","notifications_url":"https://api.github.com/repos/json-api/json-api/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/json-api/json-api/labels{/name}","releases_url":"https://api.github.com/repos/json-api/json-api/releases{/id}","created_at":"2013-05-04T19:29:30Z","updated_at":"2015-08-20T09:58:02Z","pushed_at":"2015-08-20T13:13:14Z","git_url":"git://github.com/json-api/json-api.git","ssh_url":"git@github.com:json-api/json-api.git","clone_url":"https://github.com/json-api/json-api.git","svn_url":"https://github.com/json-api/json-api","homepage":"http://jsonapi.org","size":4700,"stargazers_count":2766,"watchers_count":2766,"language":"CSS","has_issues":true,"has_downloads":true,"has_wiki":false,"has_pages":true,"forks_count":331,"mirror_url":null,"open_issues_count":55,"forks":331,"open_issues":55,"watchers":2766,"default_branch":"gh-pages"}},"_links":{"self":{"href":"https://api.github.com/repos/json-api/json-api/pulls/849"},"html":{"href":"https://github.com/json-api/json-api/pull/849"},"issue":{"href":"https://api.github.com/repos/json-api/json-api/issues/849"},"comments":{"href":"https://api.github.com/repos/json-api/json-api/issues/849/comments"},"review_comments":{"href":"https://api.github.com/repos/json-api/json-api/pulls/849/comments"},"review_comment":{"href":"https://api.github.com/repos/json-api/json-api/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/json-api/json-api/pulls/849/commits"},"statuses":{"href":"https://api.github.com/repos/json-api/json-api/statuses/421eff8d1f492b1aac9e6dfb8ae9d5db99f71759"}}}}