Super minor but I wouldn't mind getting rid of the line in between the `addSuccess()` and the `_redirect()`.  They feel like a natural "code paragraph" to me.