Skip to content
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

Added support for SendGrid and standard SMTP #625

Merged
merged 5 commits into from
Dec 26, 2020

Conversation

fegu
Copy link
Collaborator

@fegu fegu commented Dec 23, 2020

including docs

@fegu fegu requested a review from mpscholten December 23, 2020 21:36
IHP/Mail.hs Outdated Show resolved Hide resolved
IHP/Mail.hs Outdated Show resolved Hide resolved
IHP/Mail/Types.hs Outdated Show resolved Hide resolved
CONTRIBUTING.md Show resolved Hide resolved
@mpscholten
Copy link
Member

Really good addition :) Thanks!

@fegu
Copy link
Collaborator Author

fegu commented Dec 24, 2020

Merry christmas!

Copy link
Member

@mpscholten mpscholten left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perfect thanks :) have a happy christmas!

@mpscholten mpscholten merged commit febcc25 into digitallyinduced:master Dec 26, 2020
@sebastian-palma
Copy link
Contributor

There's an error when trying to compile IHP/Mail.hs (for development):

IHP/IHP/Mail.hs:22:1: error:
    Could not find module Network.Mail.SMTP
    Perhaps you meant Network.Mail.Mime (from mime-mail-0.5.0)
    Use -v (or `:set -v` in ghci) to see a list of the files searched for.
   |
22 | import qualified Network.Mail.SMTP                    as SMTP
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

Am I doing something wrong? Skipping something?

@fegu
Copy link
Collaborator Author

fegu commented Dec 27, 2020

Until the next beta is released you need to add package smtp-mail to your default.nix.

@mpscholten
Copy link
Member

Alternative you can also do this in your default.nix:

let
    ihp = ./IHP;
    haskellEnv = ...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants