Skip to content
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.

Sign up for GitHub

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

Jump to bottom

"to_user" includes self #28

Closed
vleong2332 opened this issue Jul 27, 2016 · 2 comments
Closed

"to_user" includes self #28

vleong2332 opened this issue Jul 27, 2016 · 2 comments
Labels
bug
Milestone
16.07

Comments

@vleong2332
Copy link
Contributor

vleong2332 commented Jul 27, 2016 •
edited

In initializing NewMessageForm, the field to_user is pre-populated by all users, including self.
This was referenced Jul 27, 2016
Closed
Merged
@ossanna16 ossanna16 added the bug label Jul 27, 2016
@jtauber jtauber added this to the 16.07 milestone Jul 27, 2016
@grahamu
Copy link
Contributor

grahamu commented Jul 27, 2016

I concur with @vleong2332, including the messaging user in possible recipient list is not desired.

@vleong2332
Copy link
Contributor Author

Should be resolved by PR #30

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
16.07
Development

No branches or pull requests
4 participants
@jtauber @grahamu @ossanna16 @vleong2332