It would be better to use bcrypt, beause its more secure as it's slower (uses more computing cycles).
Your code could also be better:

You wouldn't need salt field in User model, because it's saved into the same field as password does.

For authentication, something like:

var mongoose = require('mongoose'),
  bcrypt = require('bcryptjs');

var userSchema = mongoose.Schema({
  email: String,

When an agent mute/unmute the converstion, add an activity message "#{agent_name} has muted the conversation"

Is your feature request related to a problem?

When I'm filtering a view by cohort, I probably want to see how this cohort is made and I might want to edit it

Describe the solution you'd like

• When filtering something by a cohort, show a link (using the component

Is your feature request related to a problem? Please describe.
We have a lot of any types in our frontend. Let's start replacing these with more descriptive types.

Describe the solution you'd like
In all the components in assets/src/components/conversations, replace as many any types with Conversation/Message/Account/User types from https://github.com/papercups-io/paperc

Context

• This is part of release-1.5 #148
• MEDIUM priority task

Tasks

• Remove dependency bcrypt-nodejs in package.json
• Add dependency bcrypt in package.json
• Migrate file app/data/user-dao.js to bcrypt
• Validate the instalation with the local test
• Add and submit the chang