Review Board 1.7.22


Implement wave-317: Allow adding (copy/pasting) multiple participants into a wave

Review Request #3882 - Created Feb. 13, 2012 and submitted

rocklund
trunk
wave-317
Reviewers
wave
wave
Implemented as suggested in the jira-issue as a client side implementation in ParticipantController.java. I selected the separator for the multiple participant list to be comma (,) as stated in the jira issue.
Compiled and run.

* Tested to add single as well as multiple participants
* Tested to add participants both with and without @localhost
* Tested to add invalid participants

Review request changed
Updated (Feb. 23, 2012, 4:41 p.m.)
  • Implemented as suggested in the jira-issue as a client side implementation in ParticipantController.java. I selected the separator for the multiple participant list to be semi-colon(;).

    Implemented as suggested in the jira-issue as a client side implementation in ParticipantController.java. I selected the separator for the multiple participant list to be comma (,) as stated in the jira issue.
Updated according to the comments from Yuri
Ship it!
Posted (Feb. 23, 2012, 4:55 p.m.)
I'll get this committed later today.