Skip to content

Adding Mailgum mailer#130

Merged
armanist merged 4 commits into
quantum-php:masterfrom
hatik92:master
Feb 28, 2023
Merged

Adding Mailgum mailer#130
armanist merged 4 commits into
quantum-php:masterfrom
hatik92:master

Conversation

@hatik92

@hatik92 hatik92 commented Feb 13, 2023

Copy link
Copy Markdown
Collaborator

No description provided.

@hatik92 hatik92 requested a review from armanist February 13, 2023 13:53
@codecov

codecov Bot commented Feb 13, 2023

Copy link
Copy Markdown

Codecov Report

Merging #130 (34ccbad) into master (3447f84) will decrease coverage by 0.64%.
The diff coverage is 15.00%.

❗ Current head 34ccbad differs from pull request most recent head 47c0d26. Consider uploading reports for the commit 47c0d26 to get more accurate results

@@             Coverage Diff              @@
##             master     #130      +/-   ##
============================================
- Coverage     69.81%   69.18%   -0.64%     
- Complexity     1747     1754       +7     
============================================
  Files           149      151       +2     
  Lines          4436     4481      +45     
============================================
+ Hits           3097     3100       +3     
- Misses         1339     1381      +42     
Impacted Files Coverage Δ
src/Libraries/Mailer/Adapters/MailgunAdapter.php 0.00% <0.00%> (ø)
src/Libraries/Mailer/Adapters/MandrillAdapter.php 0.00% <0.00%> (ø)
src/Libraries/Mailer/Mailer.php 66.31% <25.71%> (-6.04%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

class MailgunAdapter implements MailerInterface
{
/**
* Send mail by using Sendinblue

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

SendinBlue?

->setData($data)
->start();
return true;
dd($httpClient);

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

dd() ?

return true;
dd($httpClient);
} catch (\Exception $e) {
dd($e);

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

dd() ?

'mail_trap' => true,

'smtp' => [],
'smtp' => [

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

why mail_trap moved to smtp section?

@hatik92 hatik92 requested a review from armanist February 27, 2023 11:41
@armanist armanist merged commit 603d388 into quantum-php:master Feb 28, 2023
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