{"action":"created","comment":{"url":"https://api.github.com/repos/zestedesavoir/zds-site/pulls/comments/138451590","pull_request_review_id":62249274,"id":138451590,"diff_hunk":"@@ -532,16 +591,17 @@ def post(self, request, *args, **kwargs):\n         if 'text' in request.POST:\n             form = self.get_form(self.form_class)\n \n-            if 'preview' in request.POST:\n-                if request.is_ajax():\n-                    content = render_to_response('misc/previsualization.part.html', {'text': request.POST.get('text')})\n-                    return StreamingHttpResponse(content)\n-                else:\n-                    form = self.create_form(self.form_class, **{\n-                        'text': request.POST.get('text')\n-                    })\n-            elif form.is_valid():\n+            if 'preview' not in request.POST and form.is_valid():","path":"zds/forum/views.py","position":439,"original_position":439,"commit_id":"431a8c93c2b2463f24cd5e8104d3eb079f3b5201","original_commit_id":"431a8c93c2b2463f24cd5e8104d3eb079f3b5201","user":{"login":"Situphen","id":6664636,"avatar_url":"https://avatars3.githubusercontent.com/u/6664636?v=4","gravatar_id":"","url":"https://api.github.com/users/Situphen","html_url":"https://github.com/Situphen","followers_url":"https://api.github.com/users/Situphen/followers","following_url":"https://api.github.com/users/Situphen/following{/other_user}","gists_url":"https://api.github.com/users/Situphen/gists{/gist_id}","starred_url":"https://api.github.com/users/Situphen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Situphen/subscriptions","organizations_url":"https://api.github.com/users/Situphen/orgs","repos_url":"https://api.github.com/users/Situphen/repos","events_url":"https://api.github.com/users/Situphen/events{/privacy}","received_events_url":"https://api.github.com/users/Situphen/received_events","type":"User","site_admin":false},"body":"La même chose que plus haut","created_at":"2017-09-12T20:05:30Z","updated_at":"2017-09-12T20:05:30Z","html_url":"https://github.com/zestedesavoir/zds-site/pull/4621#discussion_r138451590","pull_request_url":"https://api.github.com/repos/zestedesavoir/zds-site/pulls/4621","author_association":"CONTRIBUTOR","_links":{"self":{"href":"https://api.github.com/repos/zestedesavoir/zds-site/pulls/comments/138451590"},"html":{"href":"https://github.com/zestedesavoir/zds-site/pull/4621#discussion_r138451590"},"pull_request":{"href":"https://api.github.com/repos/zestedesavoir/zds-site/pulls/4621"}}},"pull_request":{"url":"https://api.github.com/repos/zestedesavoir/zds-site/pulls/4621","id":140432141,"html_url":"https://github.com/zestedesavoir/zds-site/pull/4621","diff_url":"https://github.com/zestedesavoir/zds-site/pull/4621.diff","patch_url":"https://github.com/zestedesavoir/zds-site/pull/4621.patch","issue_url":"https://api.github.com/repos/zestedesavoir/zds-site/issues/4621","number":4621,"state":"open","locked":false,"title":"Petit refactoring forum","user":{"login":"tleb","id":7550889,"avatar_url":"https://avatars0.githubusercontent.com/u/7550889?v=4","gravatar_id":"","url":"https://api.github.com/users/tleb","html_url":"https://github.com/tleb","followers_url":"https://api.github.com/users/tleb/followers","following_url":"https://api.github.com/users/tleb/following{/other_user}","gists_url":"https://api.github.com/users/tleb/gists{/gist_id}","starred_url":"https://api.github.com/users/tleb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tleb/subscriptions","organizations_url":"https://api.github.com/users/tleb/orgs","repos_url":"https://api.github.com/users/tleb/repos","events_url":"https://api.github.com/users/tleb/events{/privacy}","received_events_url":"https://api.github.com/users/tleb/received_events","type":"User","site_admin":false},"body":"| Q                                   | R\r\n| ----------------------------------- | -------------------------------------------\r\n| Type de modification                | refactoring\r\n| Ticket(s) (_issue(s)_) concerné(s)  | \r\n\r\n[Ca parlait de code pas très beau](https://zestedesavoir.com/billets/2127/communaute-libristes-et-projet-libre/#p162057), donc j'ai fait un tour pour voir si je pouvais aider un peu. Ce ne sont que des refactoring très légers du type, passer une condition devant pour enlever un niveau d'indentation, utiliser `str.format()`, etc. Je n'ai regardé que la partie forum. Dites moi si je peux faire quoi que ce soit pour simplifier la QA.\r\n\r\n### QA\r\n\r\n- Un peu tout le code du forum a été touché donc ça risque d'être dur à QA. Je peux faire quoi pour aider ?","created_at":"2017-09-11T20:49:17Z","updated_at":"2017-09-12T20:05:30Z","closed_at":null,"merged_at":null,"merge_commit_sha":"f2373bb6496ec996f6e307ae970457b3dc1ee5a5","assignee":null,"assignees":[],"requested_reviewers":[],"milestone":null,"commits_url":"https://api.github.com/repos/zestedesavoir/zds-site/pulls/4621/commits","review_comments_url":"https://api.github.com/repos/zestedesavoir/zds-site/pulls/4621/comments","review_comment_url":"https://api.github.com/repos/zestedesavoir/zds-site/pulls/comments{/number}","comments_url":"https://api.github.com/repos/zestedesavoir/zds-site/issues/4621/comments","statuses_url":"https://api.github.com/repos/zestedesavoir/zds-site/statuses/431a8c93c2b2463f24cd5e8104d3eb079f3b5201","head":{"label":"tleb:changes","ref":"changes","sha":"431a8c93c2b2463f24cd5e8104d3eb079f3b5201","user":{"login":"tleb","id":7550889,"avatar_url":"https://avatars0.githubusercontent.com/u/7550889?v=4","gravatar_id":"","url":"https://api.github.com/users/tleb","html_url":"https://github.com/tleb","followers_url":"https://api.github.com/users/tleb/followers","following_url":"https://api.github.com/users/tleb/following{/other_user}","gists_url":"https://api.github.com/users/tleb/gists{/gist_id}","starred_url":"https://api.github.com/users/tleb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tleb/subscriptions","organizations_url":"https://api.github.com/users/tleb/orgs","repos_url":"https://api.github.com/users/tleb/repos","events_url":"https://api.github.com/users/tleb/events{/privacy}","received_events_url":"https://api.github.com/users/tleb/received_events","type":"User","site_admin":false},"repo":{"id":80981025,"name":"zds-site","full_name":"tleb/zds-site","owner":{"login":"tleb","id":7550889,"avatar_url":"https://avatars0.githubusercontent.com/u/7550889?v=4","gravatar_id":"","url":"https://api.github.com/users/tleb","html_url":"https://github.com/tleb","followers_url":"https://api.github.com/users/tleb/followers","following_url":"https://api.github.com/users/tleb/following{/other_user}","gists_url":"https://api.github.com/users/tleb/gists{/gist_id}","starred_url":"https://api.github.com/users/tleb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tleb/subscriptions","organizations_url":"https://api.github.com/users/tleb/orgs","repos_url":"https://api.github.com/users/tleb/repos","events_url":"https://api.github.com/users/tleb/events{/privacy}","received_events_url":"https://api.github.com/users/tleb/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/tleb/zds-site","description":"Dépôt de ZdS, faisant notamment fonctionner https://zestedesavoir.com","fork":true,"url":"https://api.github.com/repos/tleb/zds-site","forks_url":"https://api.github.com/repos/tleb/zds-site/forks","keys_url":"https://api.github.com/repos/tleb/zds-site/keys{/key_id}","collaborators_url":"https://api.github.com/repos/tleb/zds-site/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/tleb/zds-site/teams","hooks_url":"https://api.github.com/repos/tleb/zds-site/hooks","issue_events_url":"https://api.github.com/repos/tleb/zds-site/issues/events{/number}","events_url":"https://api.github.com/repos/tleb/zds-site/events","assignees_url":"https://api.github.com/repos/tleb/zds-site/assignees{/user}","branches_url":"https://api.github.com/repos/tleb/zds-site/branches{/branch}","tags_url":"https://api.github.com/repos/tleb/zds-site/tags","blobs_url":"https://api.github.com/repos/tleb/zds-site/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/tleb/zds-site/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/tleb/zds-site/git/refs{/sha}","trees_url":"https://api.github.com/repos/tleb/zds-site/git/trees{/sha}","statuses_url":"https://api.github.com/repos/tleb/zds-site/statuses/{sha}","languages_url":"https://api.github.com/repos/tleb/zds-site/languages","stargazers_url":"https://api.github.com/repos/tleb/zds-site/stargazers","contributors_url":"https://api.github.com/repos/tleb/zds-site/contributors","subscribers_url":"https://api.github.com/repos/tleb/zds-site/subscribers","subscription_url":"https://api.github.com/repos/tleb/zds-site/subscription","commits_url":"https://api.github.com/repos/tleb/zds-site/commits{/sha}","git_commits_url":"https://api.github.com/repos/tleb/zds-site/git/commits{/sha}","comments_url":"https://api.github.com/repos/tleb/zds-site/comments{/number}","issue_comment_url":"https://api.github.com/repos/tleb/zds-site/issues/comments{/number}","contents_url":"https://api.github.com/repos/tleb/zds-site/contents/{+path}","compare_url":"https://api.github.com/repos/tleb/zds-site/compare/{base}...{head}","merges_url":"https://api.github.com/repos/tleb/zds-site/merges","archive_url":"https://api.github.com/repos/tleb/zds-site/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/tleb/zds-site/downloads","issues_url":"https://api.github.com/repos/tleb/zds-site/issues{/number}","pulls_url":"https://api.github.com/repos/tleb/zds-site/pulls{/number}","milestones_url":"https://api.github.com/repos/tleb/zds-site/milestones{/number}","notifications_url":"https://api.github.com/repos/tleb/zds-site/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/tleb/zds-site/labels{/name}","releases_url":"https://api.github.com/repos/tleb/zds-site/releases{/id}","deployments_url":"https://api.github.com/repos/tleb/zds-site/deployments","created_at":"2017-02-05T08:54:00Z","updated_at":"2017-02-05T08:54:05Z","pushed_at":"2017-09-12T16:25:59Z","git_url":"git://github.com/tleb/zds-site.git","ssh_url":"git@github.com:tleb/zds-site.git","clone_url":"https://github.com/tleb/zds-site.git","svn_url":"https://github.com/tleb/zds-site","homepage":"","size":107942,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"dev"}},"base":{"label":"zestedesavoir:dev","ref":"dev","sha":"0b6b498e925013abdf6d634230c5051f61c39803","user":{"login":"zestedesavoir","id":7406239,"avatar_url":"https://avatars3.githubusercontent.com/u/7406239?v=4","gravatar_id":"","url":"https://api.github.com/users/zestedesavoir","html_url":"https://github.com/zestedesavoir","followers_url":"https://api.github.com/users/zestedesavoir/followers","following_url":"https://api.github.com/users/zestedesavoir/following{/other_user}","gists_url":"https://api.github.com/users/zestedesavoir/gists{/gist_id}","starred_url":"https://api.github.com/users/zestedesavoir/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zestedesavoir/subscriptions","organizations_url":"https://api.github.com/users/zestedesavoir/orgs","repos_url":"https://api.github.com/users/zestedesavoir/repos","events_url":"https://api.github.com/users/zestedesavoir/events{/privacy}","received_events_url":"https://api.github.com/users/zestedesavoir/received_events","type":"Organization","site_admin":false},"repo":{"id":14778920,"name":"zds-site","full_name":"zestedesavoir/zds-site","owner":{"login":"zestedesavoir","id":7406239,"avatar_url":"https://avatars3.githubusercontent.com/u/7406239?v=4","gravatar_id":"","url":"https://api.github.com/users/zestedesavoir","html_url":"https://github.com/zestedesavoir","followers_url":"https://api.github.com/users/zestedesavoir/followers","following_url":"https://api.github.com/users/zestedesavoir/following{/other_user}","gists_url":"https://api.github.com/users/zestedesavoir/gists{/gist_id}","starred_url":"https://api.github.com/users/zestedesavoir/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zestedesavoir/subscriptions","organizations_url":"https://api.github.com/users/zestedesavoir/orgs","repos_url":"https://api.github.com/users/zestedesavoir/repos","events_url":"https://api.github.com/users/zestedesavoir/events{/privacy}","received_events_url":"https://api.github.com/users/zestedesavoir/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/zestedesavoir/zds-site","description":"Dépôt de Zeste de Savoir","fork":false,"url":"https://api.github.com/repos/zestedesavoir/zds-site","forks_url":"https://api.github.com/repos/zestedesavoir/zds-site/forks","keys_url":"https://api.github.com/repos/zestedesavoir/zds-site/keys{/key_id}","collaborators_url":"https://api.github.com/repos/zestedesavoir/zds-site/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/zestedesavoir/zds-site/teams","hooks_url":"https://api.github.com/repos/zestedesavoir/zds-site/hooks","issue_events_url":"https://api.github.com/repos/zestedesavoir/zds-site/issues/events{/number}","events_url":"https://api.github.com/repos/zestedesavoir/zds-site/events","assignees_url":"https://api.github.com/repos/zestedesavoir/zds-site/assignees{/user}","branches_url":"https://api.github.com/repos/zestedesavoir/zds-site/branches{/branch}","tags_url":"https://api.github.com/repos/zestedesavoir/zds-site/tags","blobs_url":"https://api.github.com/repos/zestedesavoir/zds-site/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/zestedesavoir/zds-site/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/zestedesavoir/zds-site/git/refs{/sha}","trees_url":"https://api.github.com/repos/zestedesavoir/zds-site/git/trees{/sha}","statuses_url":"https://api.github.com/repos/zestedesavoir/zds-site/statuses/{sha}","languages_url":"https://api.github.com/repos/zestedesavoir/zds-site/languages","stargazers_url":"https://api.github.com/repos/zestedesavoir/zds-site/stargazers","contributors_url":"https://api.github.com/repos/zestedesavoir/zds-site/contributors","subscribers_url":"https://api.github.com/repos/zestedesavoir/zds-site/subscribers","subscription_url":"https://api.github.com/repos/zestedesavoir/zds-site/subscription","commits_url":"https://api.github.com/repos/zestedesavoir/zds-site/commits{/sha}","git_commits_url":"https://api.github.com/repos/zestedesavoir/zds-site/git/commits{/sha}","comments_url":"https://api.github.com/repos/zestedesavoir/zds-site/comments{/number}","issue_comment_url":"https://api.github.com/repos/zestedesavoir/zds-site/issues/comments{/number}","contents_url":"https://api.github.com/repos/zestedesavoir/zds-site/contents/{+path}","compare_url":"https://api.github.com/repos/zestedesavoir/zds-site/compare/{base}...{head}","merges_url":"https://api.github.com/repos/zestedesavoir/zds-site/merges","archive_url":"https://api.github.com/repos/zestedesavoir/zds-site/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/zestedesavoir/zds-site/downloads","issues_url":"https://api.github.com/repos/zestedesavoir/zds-site/issues{/number}","pulls_url":"https://api.github.com/repos/zestedesavoir/zds-site/pulls{/number}","milestones_url":"https://api.github.com/repos/zestedesavoir/zds-site/milestones{/number}","notifications_url":"https://api.github.com/repos/zestedesavoir/zds-site/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/zestedesavoir/zds-site/labels{/name}","releases_url":"https://api.github.com/repos/zestedesavoir/zds-site/releases{/id}","deployments_url":"https://api.github.com/repos/zestedesavoir/zds-site/deployments","created_at":"2013-11-28T14:23:48Z","updated_at":"2017-09-10T13:24:08Z","pushed_at":"2017-09-12T16:52:24Z","git_url":"git://github.com/zestedesavoir/zds-site.git","ssh_url":"git@github.com:zestedesavoir/zds-site.git","clone_url":"https://github.com/zestedesavoir/zds-site.git","svn_url":"https://github.com/zestedesavoir/zds-site","homepage":"https://zestedesavoir.com","size":110952,"stargazers_count":191,"watchers_count":191,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":134,"mirror_url":null,"open_issues_count":250,"forks":134,"open_issues":250,"watchers":191,"default_branch":"dev"}},"_links":{"self":{"href":"https://api.github.com/repos/zestedesavoir/zds-site/pulls/4621"},"html":{"href":"https://github.com/zestedesavoir/zds-site/pull/4621"},"issue":{"href":"https://api.github.com/repos/zestedesavoir/zds-site/issues/4621"},"comments":{"href":"https://api.github.com/repos/zestedesavoir/zds-site/issues/4621/comments"},"review_comments":{"href":"https://api.github.com/repos/zestedesavoir/zds-site/pulls/4621/comments"},"review_comment":{"href":"https://api.github.com/repos/zestedesavoir/zds-site/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/zestedesavoir/zds-site/pulls/4621/commits"},"statuses":{"href":"https://api.github.com/repos/zestedesavoir/zds-site/statuses/431a8c93c2b2463f24cd5e8104d3eb079f3b5201"}},"author_association":"CONTRIBUTOR"}}
{"actor":{"display_login":"Situphen"}}