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

Remove fastmail.fm provider as it is legitimate #46

Closed
wants to merge 1 commit into from

Conversation

kennym
Copy link

@kennym kennym commented Dec 1, 2015

We have active users with our product with that email provider, and they're legitimate users. Also fastmail.fm doesn't offer you to create disposable emails.

Accept fastmail.fm emails
Conflicts:
	platform/ruby/mail_checker.rb
@FGRibreau
Copy link
Owner

Fixed in v1.5.1 :)

@kennym
Copy link
Author

kennym commented Dec 1, 2015

@FGRibreau thanks!

I would be cool if you could cherry-pick the changes next time so I would appear in the "Contributors" section. 😄 No big deal.

@FGRibreau
Copy link
Owner

@kennym indeed, I'm sorry! I intentionally did my own PR to remove fastmail.fm in order to be sure that no other domain were removed, what line should I add inside the "contributor" section in package.json?

@kennym
Copy link
Author

kennym commented Dec 1, 2015

I am referring to this section: https://github.com/FGRibreau/mailchecker/graphs/contributors

@FGRibreau
Copy link
Owner

Oh I see, I'm sorry, I won't be able to do this soon but if you find any other issue related to mailchecker, be sure I will merge it instantanely! I'm really sorry for the inconvenience :(

@kennym
Copy link
Author

kennym commented Dec 2, 2015

@FGRibreau no inconvenience at all! 😄

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.

2 participants