Edit response

[jackcarlisle]

ready for review

• adds the ability to edit a response to feedback after it has been made
• includes all previous commits

#61

[codecov-io]

Codecov Report

Merging #68 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff           @@
##           master    #68    +/-   ##
======================================
  Coverage     100%   100%            
======================================
  Files           5     17    +12     
  Lines          17    192   +175     
======================================
+ Hits           17    192   +175

Impacted Files	Coverage Δ
web/router.ex	100% <ø> (ø)	⬆️
test/support/conn_case.ex	100% <ø> (ø)	⬆️
web/controllers/response_controller.ex	100% <100%> (ø)
web/controllers/session_controller.ex	100% <100%> (ø)
web/controllers/feedback_controller.ex	100% <100%> (ø)	⬆️
web/models/feedback.ex	100% <100%> (ø)
web/controllers/forum_controller.ex	100% <100%> (ø)
web/models/response.ex	100% <100%> (ø)
test/support/test_helpers.ex	100% <100%> (ø)
web/controllers/auth.ex	100% <100%> (ø)
... and 17 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 df38df4...5f39aaa. Read the comment docs.

[nelsonic]

@jackcarlisle will take at look at this (mega) PR over the weekend ...

[jackcarlisle]

@nelsonic awesome thanks!

[nelsonic]

Ran locally got:

🎉

[nelsonic]

However ... getting channel error in terminal:

@jackcarlisle please create an issue to investigate this separately from this PR. 👍

[nelsonic]

Feedback left:

DB record:

[nelsonic]

@jackcarlisle this is superb! 😍
Sorry it's taken me a few days to review.
I'm quite slow at reading code
(especially so with "mega" PRs...)
and keep getting interrupted ...

But this is an excellent start to the Feedback project! 🎉
Thanks for all your hard work! ❤️ ✅

[nelsonic]

@jackcarlisle please delete any unused branches. 👍
and ... #34 ? 😉