Skip to content

Commit

Permalink
Fix some formatting for Novu. Refs: #50414
Browse files Browse the repository at this point in the history
  • Loading branch information
wouter-toppy committed May 26, 2023
1 parent 599fae3 commit 0935f1a
Show file tree
Hide file tree
Showing 3 changed files with 13 additions and 14 deletions.
4 changes: 1 addition & 3 deletions src/Symfony/Component/Notifier/Bridge/Novu/NovuTransport.php
Original file line number Diff line number Diff line change
Expand Up @@ -28,10 +28,8 @@ class NovuTransport extends AbstractTransport
{
protected const HOST = 'web.novu.co';

private string $apiKey;

public function __construct(
#[\SensitiveParameter] string $apiKey,
#[\SensitiveParameter] protected string $apiKey,
HttpClientInterface $client = null,
EventDispatcherInterface $dispatcher = null
) {
Expand Down
2 changes: 1 addition & 1 deletion src/Symfony/Component/Notifier/Bridge/Novu/README.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
Novu Notifier
=================
=============

Provides [Novu](https://novu.co/) integration for Symfony Notifier.

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,15 +32,16 @@ public function testToArray()
[],
);

$expected = [
'firstName' => 'Joe',
'lastName' => 'Smith',
'email' => 'test@example.com',
'phone' => null,
'avatar' => null,
'locale' => null,
];

$this->assertSame($expected, $options->toArray());
$this->assertSame(
[
'firstName' => 'Joe',
'lastName' => 'Smith',
'email' => 'test@example.com',
'phone' => null,
'avatar' => null,
'locale' => null,
],
$options->toArray()
);
}
}

0 comments on commit 0935f1a

Please sign in to comment.