search

BetterWorks / django-anonymizer

MIT License
14 stars 12 forks source link

Prompt confirmation of anonymization #33

Closed jjhuang closed 6 years ago

jjhuang commented 6 years ago

Since anonymizer does it in-place / is destructive, it's good to have a confirmation prompt ensuring we are anonymizing the correct DB.

@BetterWorks/engineers @terite

codecov[bot] commented 6 years ago

Codecov Report

Merging #33 into master will increase coverage by 19.6%. The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #33      +/-   ##
=========================================
+ Coverage   47.29%   66.9%   +19.6%     
=========================================
  Files           9       9              
  Lines        1960     426    -1534     
  Branches      651      73     -578     
=========================================
- Hits          927     285     -642     
+ Misses        972     130     -842     
+ Partials       61      11      -50
Impacted Files Coverage Δ
anonymizer/management/commands/anonymize_data.py 0% <0%> (ø) :arrow_up:
anonymizer/utils.py 0% <0%> (-48.4%) :arrow_down:
anonymizer/base.py 82.32% <0%> (+37.24%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 2d25bb6...88535ac. Read the comment docs.