Skip to content

[Notifier] add doc for free mobile dsn #37809

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

Merged

Conversation

noniagriconomie
Copy link
Contributor

@noniagriconomie noniagriconomie commented Aug 12, 2020

Q A
Branch? master (maybe 5.1 ? it is only code doc)
Bug fix? no
New feature? no
Deprecations? no
Tickets Fix symfony/symfony-docs#13600 (comment)
License MIT
Doc PR

Improve readme according to the linked comment
I've taken info of my original PR #35690 (with some rewrite)

Also the package was renamed, fixing the doc here symfony/symfony-docs#14057

@fabpot
Copy link
Member

fabpot commented Aug 12, 2020

Thank you @noniagriconomie.

@fabpot fabpot merged commit fc8a1ac into symfony:master Aug 12, 2020
@noniagriconomie noniagriconomie deleted the feature-free-mobile-notifier-doc branch August 12, 2020 09:51
xabbuh added a commit to symfony/symfony-docs that referenced this pull request Aug 21, 2020
This PR was submitted for the master branch but it was merged into the 5.1 branch instead.

Discussion
----------

Fix package name modified post merge

hi, small fix related to symfony/symfony@a89a2a8
Linked to symfony/symfony#37809

thx !

Commits
-------

0bf5920 Fix package name modified post merge
fabpot added a commit that referenced this pull request Aug 23, 2020
This PR was merged into the 5.1 branch.

Discussion
----------

[Notifier] backport documentation changes

| Q             | A
| ------------- | ---
| Branch?       | 5.1
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       |
| License       | MIT
| Doc PR        |

I think #37809 should have been merged into the `5.1` branch.

Commits
-------

ff768fb [Notifier] add doc for free mobile dsn
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.

3 participants