-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3882/
-----------------------------------------------------------
(Updated 2012-02-23 16:41:26.330316)
Review request for wave.
Changes
-------
Updated according to the comments from Yuri
Summary (updated)
-------
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.
This addresses bug wave-317.
https://issues.apache.org/jira/browse/wave-317
Diffs (updated)
-----
src/org/waveprotocol/wave/client/wavepanel/impl/edit/ParticipantController.java
ec9e25a
test/org/waveprotocol/wave/client/wavepanel/impl/edit/ParticipantControllerTest.java
PRE-CREATION
Diff: https://reviews.apache.org/r/3882/diff
Testing
-------
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
Thanks,
rocklund
