Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Fixup removal of duplicate user_ips rows #4432

Merged
merged 3 commits into from
Jan 23, 2019

Conversation

erikjohnston
Copy link
Member

@erikjohnston erikjohnston commented Jan 22, 2019

This includes a few changes that should hopefully make it a bit cleaner.

Now based on #4434

@codecov-io
Copy link

codecov-io commented Jan 22, 2019

Codecov Report

Merging #4432 into develop will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           develop   #4432      +/-   ##
==========================================
- Coverage     73.7%   73.7%   -0.01%     
==========================================
  Files          300     300              
  Lines        29705   29705              
  Branches      4882    4882              
==========================================
- Hits         21895   21894       -1     
+ Misses        6385    6383       -2     
- Partials      1425    1428       +3

@erikjohnston erikjohnston force-pushed the erikj/fixup_remove_old_user_ips branch 2 times, most recently from 7aac8fc to 1b3cc39 Compare January 22, 2019 13:36
@erikjohnston erikjohnston force-pushed the erikj/fixup_remove_old_user_ips branch from 1b3cc39 to 6e05f9b Compare January 22, 2019 16:51
@erikjohnston erikjohnston force-pushed the erikj/fixup_remove_old_user_ips branch from 6e05f9b to 2f947b7 Compare January 22, 2019 16:52
@hawkowl hawkowl merged commit 90743c9 into develop Jan 23, 2019
@hawkowl hawkowl deleted the erikj/fixup_remove_old_user_ips branch January 23, 2019 08:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants