search

fgrehm / letter_opener_web

A web interface for browsing Ruby on Rails sent emails
MIT License
718 stars 112 forks source link

Fix namespace #79

Closed koheisg closed 6 years ago

koheisg commented 6 years ago

This is related to #76

I have same problem. but I think https://github.com/fgrehm/letter_opener_web/pull/78 is too much.

I just fix namespace.

davekaro commented 6 years ago

FWIW this change was introduced here https://github.com/fgrehm/letter_opener_web/commit/657399f1fc0b683626d54c9e2271e799115dd0a6#diff-dc783fd3c19113d3404d8fe24d81d1bb

Not sure why the require_dependency was removed, but it is causing Rails apps to load the application's ApplicationController instead of the LetterOpener version

@pseudomuto would be nice to hear your thoughts on this. Thanks!