letter opener is a test dependency

The reason things are dev dependencies is that we don't want CI to have to install them