ponder-lab / Imperative-DL-Study-Web-App

0 stars 0 forks source link

Remove bug fix column #81

Closed khatchad closed 3 years ago

khatchad commented 3 years ago

Now that #70 has been fixed, we don't need the bug fix column on https://fathomless-inlet-57767.herokuapp.com/ponder/bug_fixes/3/. I think it's redundant. Thoughts?

mZneit commented 3 years ago

yes, I remember that I removed it before but it was re-added bcz of this commit https://github.com/ponder-lab/Imperative-DL-Study-Web-App/commit/6950a83fe5850948a6baa9042f5053b317bf05e7#diff-faecce24a655b36e6c2b3d9866dc64b19cb5f01fef881638824a1eb77f021257 The name of the field was changed from bug_fix_id to bug_fix, so it canceled the column removal command. That's why it reappeared again. But that was good since it revealed the bug in #70.

khatchad commented 3 years ago

Right, we discussed that. Thanks, @mZneit.