Closed yccheok closed 7 years ago
@luizgrp
Sorry. I think there's possible bug in my previous commits.
Do you think we need to handle case for -ve ?
If you agree with the fix, I will add more unit test.
Done with unit tests. Hope you like it :)
Just one thing on naming
I'm not sure using ContentItemsCount or ContentItemsTotal better.
ContentItemsCount
ContentItemsTotal
@luizgrp
Sorry. I think there's possible bug in my previous commits.
Do you think we need to handle case for -ve ?
If you agree with the fix, I will add more unit test.