CCI-MOC / flocx-market

2 stars 9 forks source link

Remove is_admin check from ocr update #130

Closed tzumainn closed 5 years ago

tzumainn commented 5 years ago

The provider needs to be able to update an ocr, but the is_admin check prevents it from doing so.

Fixes #126

codecov-io commented 5 years ago

Codecov Report

Merging #130 into master will decrease coverage by 0.46%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #130      +/-   ##
==========================================
- Coverage   93.66%   93.19%   -0.47%     
==========================================
  Files          51       51              
  Lines        1972     1970       -2     
==========================================
- Hits         1847     1836      -11     
- Misses        125      134       +9
Impacted Files Coverage Δ
flocx_market/db/sqlalchemy/api.py 95.75% <100%> (-0.06%) :arrow_down:
flocx_market/tests/unit/db/sqlalchemy/test_api.py 96.59% <100%> (-3.41%) :arrow_down:

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 82e3170...f779b48. Read the comment docs.