Closed dadak-dom closed 5 months ago
Good find, @dadak-dom!
I believe we had an earlier conversation about this point, though, I am not 100% sure if it relates to the same point. You can check the issue here. I am good with your change, especially, since it decreases false positives. But also take a look at the previous issue and check if it is related and your change does not revert back something we had considered problematic (or your point is more important).
Good point, @SebastianZimmeck. I believe my changes are in line with what Joe was going for, since they don't remove the ability for the user to set a null address/region/zip code 👍 .
Sounds good, @dadak-dom!
Can you review this PR, @atlasharry?
Sorry @dadak-dom @SebastianZimmeck , I did not receive this notification until I manually browsed this PR. I will double-check my settings and make sure I will not miss your message again. I will go through your PR right now!
Sorry @dadak-dom @SebastianZimmeck , I did not receive this notification until I manually browsed this PR. I will double-check my settings and make sure I will not miss your message again. I will go through your PR right now!
No worries!
While I was poking around with testing our web crawler, I discovered a bug where, if a user decided to input a Street Address to their watchlist that didn't include a region or zip code (i.e., they only input the city, which is the bare minimum for an entry), then the extension would get flooded with false-positive results, like so:
Thankfully, I just had to make sure that empty regions and zip codes weren't being added to the list of keywords to look out for. I've also included some fixes for how the watchlist is displayed when only a city is the input to the Street Address. @atlasharry, I think this is a good chance for you to take a first look at the extension code. I've done a fair amount of testing on these changes and I'm fairly confident they'll work, but I'd like for someone else to check just in case.