Closed jaredpetersen closed 9 years ago
Ooh, I like that! Good choice on the highlighting!
Sorry, I had to revert these changes because they broke searching Github repositories. You can use the Servo
category as a good testcase.
Hmmm... Sorry about that. I'll look into it as soon as I can.
I'm testing a fork with the diamond bugs added back in, and I'm not seeing searches on GitHub repositories breaking. What was the exact symptopm?
Here are my test cases:
Test 1 -> "Servo" Checked Expected result: Only servo bugs appear Actual result: Only servo bugs appear
Test 2 -> "Servo" and "Diamond Bugs" Checked Expected result: No bugs shown, GitHub doesn't have diamond bugs Actual result: No bugs shown, GitHub doesn't have diamond bugs
Test 3 -> "Diamond Bugs" Checked Expected result: Only diamond bugs shown Actual result: Only diamond bugs are shown
You can check out the fork here https://github.com/jaredpetersen/bugsahoy.
It was the highlighting that hit the problem: https://github.com/jdm/bugsahoy/commit/f7c55425f50051921f450d1a33971df4b467f5dc . The webconsole shows "bug.whiteboard is undefined" and no github results are retrieved.
@jdm That was super helpful! I didn't implement highlighting in the fork at that point, so I wasn't seeing the problem. I've created a new pull request (#136) to reimplement diamond bugs and fix diamond bug highlighting.
Added diamond bugs and diamond bug highlighting. Let me know if you would like to use a different color. Fixes Issue #94.