adonisjs / mail

AdonisJS Email Provider
MIT License
106 stars 34 forks source link

Allow set SMTP_PORT on config #30

Closed caiquecastro closed 5 years ago

caiquecastro commented 5 years ago

Proposed changes

It assumes the port is 2525, but if I have a local server for development and a production SMTP server with different ports, it's needed to be configurable

Types of changes

What types of changes does your code introduce?

Put an x in the boxes that apply

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

Further comments

N/A

coveralls commented 5 years ago

Coverage Status

Coverage decreased (-8.9%) to 84.211% when pulling 32cf6e22c4c452ff15385179d389cd556d1bcbf9 on caiquecastro:patch-1 into 4aef9bbdb200be41f6f4e3d429d0cd1e75e736a9 on adonisjs:develop.