New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
filter new orange factor bot email #904
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
two minor changes, any vs. grep is good practice even for small lists.
if (that.data('shown') === '1') { | ||
that.data('shown', '0'); | ||
item.text('Show CC Changes'); | ||
that.text('Show CC Changes'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What did this fix?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When you clicked on the button before it wouldn't toggle the text, it would be stuck on 'Show CC Changes'. Same thing with the Show/Hide Treeherder changes, thought I'd might as well make the quick fix while I was nearby
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make sure that's in the commit message (not in the summary, just in the body).
(as for the summary, I'm fine with "no bug - correctly collapse orange factor bot comments")
Fixes filtering the orangefactor bot, before only the old treeherder bot was being filtered. (I guess this has been a bug for years)