-
Notifications
You must be signed in to change notification settings - Fork 443
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
Find email duplicates API #3828
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please fix the SQL statement character case
src/api/routes/email.php
Outdated
function getEmailDupes(Request $request, Response $response, array $p_args) | ||
{ | ||
$connection = Propel::getConnection(); | ||
$dupEmailsSQL = "select email from emails_duplicate;"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please make the SQL statements CAPITALIZED to keep with standard SQL convention.
This is for 2.10.0, since it has a DB script, correct? |
correct, this is 2.10.0 |
looks like i missed adding to view on new install.. adding that back |
…M/CRM into feature/find-email-dups
in testing, I noticed our vagrant is running MySQL 5.7.20 the way the create view is done does not work pre - MySQL 5.7.7 |
now it is ready |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't publish your mailtrap key in the demo DB
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok. It looks like these credentials were already in the demo DB before this PR, so I'll approve
What's this PR do?
What Issues does it Close?
part of #3787
Test via
http://192.168.33.10/api/emails/duplicates