search

stxnext-csr / volontulo

Web portal for collaboration of community volunteers with organizations and institutions.
MIT License
14 stars 24 forks source link

Closes #450 and #451 #507

Closed KrzysztofMadejski closed 8 years ago

KrzysztofMadejski commented 8 years ago

Story / Bug id: https://github.com/stxnext-csr/volontulo/issues/450 Story / Bug id: https://github.com/stxnext-csr/volontulo/issues/451 Reference person: @krzysztofmadejski Description:

Unit Tests:

What tests do I need to run to validate this change:

KrzysztofMadejski commented 8 years ago

Needs a bit of love on the design side.

codecov-io commented 8 years ago

Current coverage is 90.79%

Merging #507 into master will increase coverage by +0.02% as of ee68eb5

@@            master    #507   diff @@
======================================
  Files           41      41       
  Stmts         1875    1880     +5
  Branches         0       0       
  Methods          0       0       
======================================
+ Hit           1702    1707     +5
  Partial          0       0       
  Missed         173     173

Review entire Coverage Diff as of ee68eb5

Powered by Codecov. Updated on successful CI builds.

tomaszwozniak commented 8 years ago

Unnecessary u" but I won't block this PR. Rest looks good

tomaszwozniak commented 8 years ago

@KrzysztofMadejski please update the branch

msobas commented 8 years ago

:+1:

KrzysztofMadejski commented 8 years ago

@magul prośba o merge