breachDateThreshold is a confusing variable name

mozilla / blurts-addon

Mozilla Public License 2.0

5 stars 8 forks source link

Open groovecoder opened 5 years ago

groovecoder commented 5 years ago

The code uses breachDateThreshold, but the comparison logic (correctly) uses the breach AddedDate.

Might be clearer to re-name breachDateThreshold to breachAddedDateThreshold.

tcinotto commented 5 years ago

Target for 68.