Skip to content

Conversation

@magdalenesuo
Copy link
Member

fixed #2978
Wrong Sales on Admin:
screenshot from 2017-01-30 23-30-56

Corrected Sales on admin:
screenshot from 2017-01-30 23-31-36

@codecov-io
Copy link

codecov-io commented Jan 30, 2017

Codecov Report

Merging #3090 into development will increase coverage by -0.11%.

@@              Coverage Diff               @@
##           development   #3090      +/-   ##
==============================================
- Coverage        76.11%     76%   -0.11%     
==============================================
  Files              222     222              
  Lines            14806   14806              
==============================================
- Hits             11269   11254      -15     
- Misses            3537    3552      +15
Impacted Files Coverage Δ
app/views/super_admin/sales.py 18.67% <ø> (ø)
app/helpers/deployment/heroku.py 59.09% <ø> (-27.28%)
app/views/super_admin/super_admin.py 70.9% <ø> (-12.73%)
app/helpers/helpers.py 48.97% <ø> (-0.69%)

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 7b06621...4521f11. Read the comment docs.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.1%) to 75.774% when pulling 4521f11 on magdalenesuo:admin-sales-fix into 7b06621 on fossasia:development.

@SaptakS SaptakS merged commit c7ba812 into fossasia:development Jan 30, 2017
@mariobehling
Copy link
Member

Thanks! Haven't checked yet, if it works. The screenshots in the description (#3090 (comment)) seem to be the same, but let's try it out.

@SaptakS
Copy link
Contributor

SaptakS commented Jan 30, 2017

@mariobehling the screenshots are different. Check the pending column last row. The second one seems correct.

@mariobehling
Copy link
Member

Yeah, see it. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Overview of ticket income shows incorrect number

5 participants