Chris,
Although I have not tried to use this new functionality, I believe it is 
a needed addition to Jetspeed.  Thank you.

I have the following request:
1) For consistency please change the following parameter names:
   password.reminder.option.enable -> password.reminder.enable
   password.reminder.format -> password.reminder.template

2) In top.vm
    o remove "/" from <input... /> and <br/>. It is non-standard HTML
    o Localize the strings "Forgotten your password?"
      and "Enter your username<br> and click Password?"

3) Please remove the commented out text added in you patch to
    JLoginUser.java.

4) Add an entry to the change log (xdocs\changes.xml)

Paul Spencer

[EMAIL PROTECTED] wrote:

> kimptoc     01/12/27 07:11:11
> 
>   Modified:    src/java/org/apache/jetspeed/modules/actions JLoginUser.java
>                src/java/org/apache/jetspeed/modules/localization
>                         JetspeedLocalization_en.properties
>                webapp/WEB-INF/conf JetspeedResources.properties
>                webapp/WEB-INF/templates/vm/navigations/html top.vm
>   Log:
>   basic password reminder functionality, ala bugzilla item#4142
>   
>   Revision  Changes    Path
>   1.19      +104 -1    
>jakarta-jetspeed/src/java/org/apache/jetspeed/modules/actions/JLoginUser.java
>   



--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to