Skip to content

[Messenger] doc bloc moved #11827

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

Closed
wants to merge 1 commit into from

Conversation

noniagriconomie
Copy link
Contributor

I think moving a bit this bloc about HandleTrait is better for readability
Feel free to close the PR if not :)

@noniagriconomie noniagriconomie changed the title [Messenger] [Messenger] doc bloc moved Jun 26, 2019
@wouterj wouterj requested a review from weaverryan June 27, 2019 07:50
Copy link
Member

@weaverryan weaverryan left a comment

Choose a reason for hiding this comment

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

I added a recommended re-wording (since we’re already moving this text). But yes, I think it makes more sense to talk about this down here where we actually show it in code.

@weaverryan
Copy link
Member

Status: needs work

@noniagriconomie
Copy link
Contributor Author

noniagriconomie commented Jul 24, 2019

@@weaverryan friendly ping, can you check #11827 (comment)? thx

edit: Rewording done, need re review

@javiereguiluz
Copy link
Member

@noniagriconomie thanks for this contribution! I tried to merge it, but our private merging tool showed some conflicts ... and sent some change to your pull request (that's why you see all those modified files ... which were sent accidentally by me via this merging tool).

I created #12260 to replace this and I tried to cherry-pick your commit, but I saw the same conflict messages, so at the end I recreated the changes myself. I'm sorry!

@noniagriconomie
Copy link
Contributor Author

@javiereguiluz understood 👍

@noniagriconomie noniagriconomie deleted the patch-1 branch September 2, 2019 14:36
javiereguiluz added a commit that referenced this pull request Sep 2, 2019
…iluz)

This PR was squashed before being merged into the 4.3 branch (closes #12260).

Discussion
----------

Move some doc contents in a Messenger article

This continues #11827. All credit goes to @noniagriconomie.

Commits
-------

9707b69 Move some doc contents in a Messenger article
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.

4 participants