isn't it weird to have 'none' for the user_hash? Shouldn't this be `None` instead? maybe should have a default value for user_hash then or make it optional