Skip to content

Conversation

@noniagriconomie
Copy link
Contributor

Fix #15571

@OskarStark I am unsure if I should add a new line to the table or keep like I added, both DSN possible values
Feel free to comment :)

@carsonbot carsonbot changed the title Document fake sms logger notifier [Notifier] Document fake sms logger notifier Aug 7, 2021
@OskarStark OskarStark added this to the 5.4 milestone Aug 7, 2021
Copy link
Contributor

@OskarStark OskarStark left a comment

Choose a reason for hiding this comment

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

I am fine with this 👍

@OskarStark
Copy link
Contributor

Thanks Antoine for working on this feature, this is much appreciated.

@OskarStark OskarStark merged commit e69555d into symfony:5.4 Aug 7, 2021
OskarStark added a commit that referenced this pull request Aug 7, 2021
…iconomie)

This PR was merged into the 5.4 branch.

Discussion
----------

[Notifier] Document fake chat logger notifier

Document symfony/symfony#42188 (no issue created post merge)
Follows #15595

Commits
-------

4e5845b Document fake chat logger notifier
@noniagriconomie noniagriconomie deleted the ft-notifier-fake branch August 7, 2021 11:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Notifier] Add FakeSMS Logger transport

3 participants