Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Restore guard to populateHistory, fixes #8767 #8768

Merged
merged 1 commit into from
May 9, 2017
Merged

Restore guard to populateHistory, fixes #8767 #8768

merged 1 commit into from
May 9, 2017

Conversation

evq
Copy link
Member

@evq evq commented May 8, 2017 •
edited by bsclifton
Loading

Test Plan:

Run tests

Description

Restores guard behavior from 4ddf839#diff-303f0b6a297506f2cc18bf7b4cb370c5R844

Auditors:
@ayumi
@bsclifton

  • Submitted a ticket for my issue if one did not already exist.
  • Used Github auto-closing keywords in the commit message.
  • Added/updated tests for this change (for new code or code which already has tests).
  • Ran git rebase -i to squash commits (if needed).

@ayumi
Copy link
Contributor

ayumi commented May 9, 2017

++

@ayumi ayumi merged commit eb57069 into brave:master May 9, 2017
@evq evq deleted the fix/populate-history-spam branch May 9, 2017 00:14
@ayumi ayumi mentioned this pull request May 9, 2017
Closed
@bsclifton
Copy link
Member

Awesome, thanks @evq! 😄

@luixxiul luixxiul added this to the 0.15.3 milestone May 9, 2017
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Reviewers
No reviews
Assignees
No one assigned
Labels
automated-tests feature/history perf QA/no-qa-needed
Projects
None yet
Milestone
0.15.300 (Frozen, only critical adds ...
Development

Successfully merging this pull request may close these issues.
4 participants
@evq @ayumi @bsclifton @luixxiul