-
-
Notifications
You must be signed in to change notification settings - Fork 260
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
Migrated internal code to use Direct Message terminology #1406
base: main
Are you sure you want to change the base?
Conversation
…with DirectComposition and DirectMessageUpdateRequest
Hello @WladRamos, it seems like you have referenced #1344 in your pull request description, but you have not referenced them in your commit message description(s). Referencing an issue in a commit message automatically closes the corresponding issue when the commit is merged, which makes the issue tracker easier to manage. Please run An example of a correctly-formatted commit:
To learn how to write a great commit message, please refer to our guide. |
Heads up @WladRamos, we just merged some commits that conflict with the changes you made in this pull request! You can review this repository's recent commits to see where the conflicts occur. Please rebase your feature branch against the |
What does this PR do, and why?
This PR changes all appearances of private, pm and "private" type to Direct message terminology
fixes #1344
External discussion & connections
topic
How did you test this?
Self-review checklist for each commit