Skip to content

Commit

Permalink
Fixed checklist copy for issues
Browse files Browse the repository at this point in the history
  • Loading branch information
mjasikowski committed Jan 15, 2025
1 parent 59ccfab commit 6eee628
Showing 1 changed file with 3 additions and 18 deletions.
21 changes: 3 additions & 18 deletions src/js/lib/pages/github/issue.js
Original file line number Diff line number Diff line change
Expand Up @@ -22,23 +22,6 @@ function catchError(e) {
}, 30000);
}

/**
* Gets the contents of the reviewer checklist from GitHub and then posts it as a comment to the current PR
* @param {Event} e
*/
const copyReviewerChecklist = (e) => {
e.preventDefault();
const pathToChecklist = 'https://raw.githubusercontent.com/Expensify/App/main/contributingGuides/BUGZERO_CHECKLIST.md';
$.get(pathToChecklist)
.done((fileContents) => {
if (!fileContents) {
console.error(`could not load contents of ${pathToChecklist} for some reason`);
return;
}

API.addComment(fileContents);
});
};

Check failure on line 25 in src/js/lib/pages/github/issue.js

View workflow job for this annotation

GitHub Actions / lint

More than 1 blank line not allowed
/**
* Sets the owner of an issue when it doesn't have an owner yet
Expand Down Expand Up @@ -175,8 +158,10 @@ export default function () {
}
}, 1000);

renderLinksInTitle();

Check failure on line 161 in src/js/lib/pages/github/issue.js

View workflow job for this annotation

GitHub Actions / lint

'renderLinksInTitle' is not defined

// Waiting 2 seconds to call this gives the page enough time to load so that there is a better chance that all the comments will be rendered
setInterval(() => IssuePage.renderCopyChecklistButtons(copyReviewerChecklist), 2000);
setInterval(() => IssuePage.renderCopyChecklistButtons('bugzero'), 2000);
};

return IssuePage;
Expand Down

0 comments on commit 6eee628

Please sign in to comment.