-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/4192/
-----------------------------------------------------------

(Updated 2012-03-06 17:15:36.429202)


Review request for rave and Raminder Singh.


Summary
-------

I'm new to Rave and have been trying to get the vanilla portal up and running 
and was hung up on this issue. Attached is my patch to get things working 
better. It fixes this issue by adding the javax.mail dependency to the portal, 
fixes an issue where the DefaultUserService constructor has changed and updated 
the poms to map to version 0.10-SNAPSHOT. 

If we need to split the patch and create some other JIRA issues please let me 
know.


This addresses bug RAVE-488.
    https://issues.apache.org/jira/browse/RAVE-488


Diffs
-----

  /sandbox/rave-extensions/rave-vanilla-extension/pom.xml 1297337 
  
/sandbox/rave-extensions/rave-vanilla-extension/rave-vanilla-extension-portal/pom.xml
 1297337 
  
/sandbox/rave-extensions/rave-vanilla-extension/rave-vanilla-extension-portal/src/main/java/org/apache/rave/portal/service/impl/CustomUserService.java
 1297337 
  
/sandbox/rave-extensions/rave-vanilla-extension/rave-vanilla-extension-portal/src/test/java/org/apache/rave/portal/service/impl/CustomUserServiceTest.java
 1297337 
  
/sandbox/rave-extensions/rave-vanilla-extension/rave-vanilla-extension-shindig/pom.xml
 1297337 

Diff: https://reviews.apache.org/r/4192/diff


Testing
-------

Project will compile and deploy. There is still an error when navigating to the 
site because there is no login.jsp at the moment.


Thanks,

Chris

Reply via email to