You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@rave.apache.org by marijan milicevic <m....@onehippo.com> on 2012/01/16 20:57:36 UTC
Review Request: fix for issue RAVE-424
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3511/
-----------------------------------------------------------
Review request for rave.
Summary
-------
patch fixing user role assigning
Diffs
-----
https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-components/rave-web/src/main/java/org/apache/rave/portal/web/controller/admin/UserController.java 1232128
https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-components/rave-web/src/main/java/org/apache/rave/portal/web/validator/UserProfileValidator.java 1232128
Diff: https://reviews.apache.org/r/3511/diff
Testing
-------
Thanks,
marijan
Re: Review Request: fix for issue RAVE-424
Posted by Jasha Joachimsthal <ja...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3511/#review4404
-----------------------------------------------------------
Ship it!
Already applied it (RAVE-424)
- Jasha
On 2012-01-16 19:57:36, marijan milicevic wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/3511/
> -----------------------------------------------------------
>
> (Updated 2012-01-16 19:57:36)
>
>
> Review request for rave.
>
>
> Summary
> -------
>
> patch fixing user role assigning
>
>
> Diffs
> -----
>
> https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-components/rave-web/src/main/java/org/apache/rave/portal/web/controller/admin/UserController.java 1232128
> https://svn.apache.org/repos/asf/incubator/rave/trunk/rave-components/rave-web/src/main/java/org/apache/rave/portal/web/validator/UserProfileValidator.java 1232128
>
> Diff: https://reviews.apache.org/r/3511/diff
>
>
> Testing
> -------
>
>
> Thanks,
>
> marijan
>
>