-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
[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
[Messenger] doc bloc moved #11827
Conversation
There was a problem hiding this 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.
Status: needs work |
@@weaverryan friendly ping, can you check #11827 (comment)? thx edit: Rewording done, need re review |
48d6568
to
55be0a1
Compare
@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! |
@javiereguiluz understood 👍 |
…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
I think moving a bit this bloc about
HandleTrait
is better for readabilityFeel free to close the PR if not :)