com web/qa: Make sure we don't multiple event handlers an the "Next" link: pulls/pullrequests.js
| From: | Johannes Schlüter | Date: | Mon, 02 May 2016 19:51:57 +0000 |
| Subject: | com web/qa: Make sure we don't multiple event handlers an the "Next" link: pulls/pullrequests.js | ||
| Groups: | php.qa | ||
| Request: | Send a blank email to php-qa+get-67715@lists.php.net to get a copy of this message | ||
Commit: e45cb55866704283d48d821bd2e66908444ea068
Author: Johannes Schlüter <johannes@schlueters.de> Mon, 2 May 2016 21:51:57 +0200
Parents: 9669de2068f98f97b4820a3b2605ef224de88d15
Branches: master
Link: http://git.php.net/?p=web/qa.git;a=commitdiff;h=e45cb55866704283d48d821bd2e66908444ea068
Log:
Make sure we don't multiple event handlers an the "Next" link
This caused additional requests to github, so the rate limit was
hit quite easily when browsing for older requests
Changed paths:
M pulls/pullrequests.js
Diff:
diff --git a/pulls/pullrequests.js b/pulls/pullrequests.js
index 59baea0..8500b4a 100644
--- a/pulls/pullrequests.js
+++ b/pulls/pullrequests.js
@@ -115,6 +115,7 @@ $(document).ready(function() {
});
function loadRepo(repo, url) {
+ $("#nextRepoPage").off();
$("#loading").show();
url = url ||
GITHUB_BASEURL+'repos/'+GITHUB_ORG+"/"+repo+"/pulls";
$.ajax({