Only users without sync are eligible

mozilla / security-advisor-shield-study

Mozilla Public License 2.0

2 stars 7 forks source link

Only users without sync are eligible #38

Closed casebenton closed 7 years ago

casebenton commented 8 years ago

@Osmose r?

Osmose commented 8 years ago

Besides the things I mentioned, I think the code looks fine, but I'm blocked from r+ing due to #43 (and the commented issues).

brittanydionigi commented 7 years ago

This works as expected, but I would work through some of the feedback from @Osmose. Those would be worthwhile changes to make. I can manually review and test again after that.

casebenton commented 7 years ago

@brittanystoroz I've made the changes that @Osmose suggested.

brittanydionigi commented 7 years ago