From a modularity / code reuse standpoint, I'm wondering if we should have a separate attachment.rb model (or file.rb) that the comment.rb model uses.  Maybe we only use attachments on files for now so its fine to start with this, but we need to make sure we immediately refactor to share logic if we allow attachments to be added to other elements, like applications to Beyond Z in the recruitment user flow.