Merge pull request #162 from Expensify/maria-fix-pr-checklist

Fix for button to add PR checklist to pull requests

CHANGELOG.md

@@ -1,3 +1,6 @@
+#1.2.13
+- Improve the regex that fetches the repository name, owner and issue number from the current URL work for pull requests as well
+
 #1.2.12
 - Fetch repository name, owner and issue number from URL instead of relying on the DOM
 

assets/manifest.json

@@ -2,7 +2,7 @@
   "manifest_version": 2,
 
   "name": "K2 for GitHub",
-  "version": "1.2.12",
+  "version": "1.2.13",
   "description": "Manage your Kernel Scheduling from directly inside GitHub",
 
   "browser_specific_settings": {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "k2-extension",
-  "version": "1.2.12",
+  "version": "1.2.13",
   "description": "A Chrome Extension for Kernel Schedule",
   "private": true,
   "scripts": {

src/js/lib/api.js

@@ -28,12 +28,14 @@ function getCurrentUser() {
 
 /**
  * Returns the name of the repository, the repo owner and the issue number from the current url for calls to the Github API
+ * Issue URLs are in the format: github.com/<repoName>/<owner>/issues/<issue-number> Example: https://github.com/Expensify/App/issues/16640
+ * PR URLs are in the format: github.com/<repoName>/<owner>/pulls/<issue-number> Example: https://github.com/Expensify/App/pull/21242
  *
  * @returns {Object}
  */
 function getRequestParams() {
     const url = window.location.href;
-    const regex = /github.com\/(\w*)\/(\w*)\/issues\/(\d*)/;
+    const regex = /github.com\/(\w*)\/(\w*)\/(?:issues|pull)\/(\d*)/;
     const matches = url.match(regex);
 
     return {