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

Problem
When deleting a conversation in the database, we get an error if associated slack_conversation_threads exist.

This is because when we created the slack_conversation_threads table, we forgot to add an on_delete: :delete_all to the migration script.

This is what we did: https://github.com/papercups-io/papercups/blob/master/priv/repo/migrations/20200720153026_create_slack_c

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