From 1785e7ac64c93a8de099dec581d4a1109b46d74c Mon Sep 17 00:00:00 2001
From: Pierre-Yves Chibon <pingou@pingoured.fr>
Date: Sep 07 2017 08:47:11 +0000
Subject: Ask the user to confirm merging a pull-request


Fixes https://pagure.io/pagure/issue/2537
Fixes https://pagure.io/pagure/issue/2570

Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>

---

diff --git a/pagure/templates/pull_request.html b/pagure/templates/pull_request.html
index d557964..877915a 100644
--- a/pagure/templates/pull_request.html
+++ b/pagure/templates/pull_request.html
@@ -632,7 +632,9 @@
                 requestid=requestid)
           }}" method="POST">
           {{ mergeform.csrf_token }}
-          <button id="merge_btn" type="submit" class="btn btn-block">Merge</button>
+          <button id="merge_btn" type="submit"
+            onclick="return confirm('Confirm merging this pull-request');"
+            class="btn btn-block">Merge</button>
         </form>
        </div>
       {% endif %}