-
Notifications
You must be signed in to change notification settings - Fork 237
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Disabling Launchy #173
Comments
Hey @prwhitehead. Please see this discussion regarding this issue #117. Hope it'll work for you! |
Hey @nashby, Thank you for getting back to me. Unfortunately, the recommendations in the thread you linked are to toggle the
The point I am trying to get to is where emails are sent by my app, they are caught by letter opener and available for me to see in the interface, but that the Is this possible? Thank you for your time, |
@prwhitehead hey! Yeah, I think you can do it by simply setting ENV variable |
Hello @nashby, thank you for your response. Unfortunately, that didn't work. I am still getting the following error:
I confirmed that the ENV variable I did some googling and saw that another project seems to catch the error by monkey patching your I would prefer not to monkey patch as it means future updates and bug fixes might be missed, but is this something that you would consider adding, in order to help users that are using docker for their apps - an increasing segment of the community i'm sure. Thank you again for your help with this. |
@prwhitehead Hey! Right, so as I can see Launchy just can't find any browser since you don't have any in your docker env. Could you please try to set |
OMG YES! Thanks @nashby, thats working now. I get the following results:
If I made a PR to edit the readme would you accept that, as it might be helpful for others as more move to dockerize their apps? |
@prwhitehead awesome! Good idea about PR, go ahead, thanks! |
|
Since I have an env var |
Hello,
First of all, thank you for the Gem, its super useful.
I am trying to move an app to Docker, but LetterOpener doesn't seem to want to let me disable Launchy. I've added an initializer as the docs recommend, trying to just
nil
theLaunchy.application
however that didn't seem to have any effect.I had a look through the code and couldn't see any way to disable it within the Gem, so I was wondering if you might consider adding a flag to disable that part of it. The ability to catch emails in development is amazing, but launching a web browser/tab is something that i'd love to suppress - and instead just view the email inside of the web interface.
As you can see from the error below, my Sidekiq container is trying to send the email via ActionMailer but a
Launchy::CommandNotFoundError
error is being thrown.The text was updated successfully, but these errors were encountered: