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.

Sign up for GitHub

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

Jump to bottom

Add a way to verify that the content doesn't contain sensitive information #275

Merged
merged 3 commits into from
Dec 21, 2017
Merged

Add a way to verify that the content doesn't contain sensitive information #275

merged 3 commits into from
Dec 21, 2017

Conversation

diegoc-am
Copy link
Contributor

No description provided.

@thinkingserious thinkingserious added the status: code review request requesting a community code review or review from Twilio label Oct 16, 2017
@SendGridDX
Copy link
Collaborator

SendGridDX commented Oct 16, 2017 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@diegoc-am diegoc-am mentioned this pull request Oct 16, 2017
Closed
@thinkingserious thinkingserious added difficulty: medium fix is medium in difficulty hacktoberfest labels Oct 16, 2017
@thinkingserious
Copy link
Contributor

@belfazt

We have not been able to merge your Pull Request, but because you are awesome - we wanted to make sure you could still get a SendGrid Hacktoberfest shirt.

Please go fill out our swag form before Nov 5th and we will send the shirt! (We know that you might have tried this before and it didn’t work, sorry about that!)

You have till Nov 5th to fill out this form in order to get the Hacktoberfest shirt!

Thank you for contributing during Hacktoberfest! We hope to see you in the repos soon! Just so you know, we always give away a SendGrid shirt for your first ever non-Hacktoberfest PR that gets merged.

@diegoc-am
Copy link
Contributor Author

I usually contribute to your repos guys :)

@thinkingserious
Copy link
Contributor

Hi @belfazt,

We want to make sure you also get your limited edition Hacktoberfest swag too :)

@thinkingserious thinkingserious merged commit 0b7f335 into sendgrid:master Dec 21, 2017
@thinkingserious
Copy link
Contributor

Hello @belfazt,

Thanks again for the PR!

We appreciate your contribution and look forward to continued collaboration. Thanks!

Team SendGrid DX

@diegoc-am diegoc-am deleted the add-a-verification-before-adding-the-content-to-the-mail-issue-269 branch March 18, 2018 13:45
@childish-sambino childish-sambino mentioned this pull request Jul 24, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
difficulty: medium fix is medium in difficulty status: code review request requesting a community code review or review from Twilio
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@diegoc-am @SendGridDX @thinkingserious @apigirl @mbernier