jackcarlisle commented 7 years ago

ready for review after #56 (needs rebase)

[x] adds responded_at field to feedback table
[x] removes responded field from feedback as it was surplus to requirements

58

codecov-io commented 7 years ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@5a68388). Click here to learn what that means. The diff coverage is 100%.

@@           Coverage Diff           @@
##             master    #59   +/-   ##
=======================================
  Coverage          ?   100%           
=======================================
  Files             ?     15           
  Lines             ?    169           
  Branches          ?      0           
=======================================
  Hits              ?    169           
  Misses            ?      0           
  Partials          ?      0

Impacted Files	Coverage Δ
test/support/conn_case.ex	`100% <ø> (ø)`
web/router.ex	`100% <ø> (ø)`
web/controllers/helpers.ex	`100% <100%> (ø)`
web/views/feedback_view.ex	`100% <100%> (ø)`
test/support/test_helpers.ex	`100% <100%> (ø)`
web/models/feedback.ex	`100% <100%> (ø)`
web/controllers/page_controller.ex	`100% <100%> (ø)`
web/channels/room_channel.ex	`100% <100%> (ø)`
web/models/user.ex	`100% <100%> (ø)`
web/controllers/session_controller.ex	`100% <100%> (ø)`
... and 4 more

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 5a68388...c3ff010. Read the comment docs.

jackcarlisle commented 7 years ago

Closing due to changes in the db that are effecting tests, thus previous branches can't be merged

dwyl / feedback

Responded at #59

ready for review after #56 (needs rebase)

58

Codecov Report