portals-jetspeed-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Chris Kimpton <kimptoc_m...@yahoo.com>
Subject [Bug 4192] - Manual approval of new users under jetspeed
Date Wed, 31 Oct 2001 14:38:00 GMT
> 
> ------- Additional Comments From paulsp@apache.org  2001-10-31
> 05:18 -------
> Chris,
> The following item need to be addressed:

fine.

> 
> o Language of approver and new user.
>   Jetspeed must maintain multi-language support.  In this case we
>   cannot assume the "approver" speaks the "new user's" language,
> and 
>   vice versa.
>   Suggest you add "new.user.notification.language=" to JR.p.  On
> the
>   user side, see below.
> 

The problem with this and the one below is that we need to select a
template based on a specified locale - the current code assumes that
its the users own locale.

Thus I suggest we add the locale to the locateEmailTemplate method -
since its only being added for this reason.  

Is this ok?

I will do a revised patch and update the email templates bug item.

> o One of two possible email messages will be sent, but their is
> only
>   configuration for one.
>   Suggest you add configuration to include:
>     o Name and Subject of "notification of new user" email 
>     o Name and Subject of "need approval for a new user" email 

I assumed that only one of the two will be sent - and thus only need
configuration for one.  The on/off flags indicate how they will be
used.

Do you want the option of separate notifications of new users from
the approval emails?  If so, I presume these would be sent upon the
user being accepted?

I'd prefer not to do this at this stage;  a) cos I don't need it,  b)
the change is big enough already that I want to minimise the initial
patch for it.  We can then refine it.


>   
> o How will the user know when he can login?
>   Suggest you add an email notification the send/reject button. 
> The
>   email must be the "new user's" language.  You should be able to
>   accomplish this either by adding the columns Language and Country
>   ( more work, but preferred ) or including Language and Country in
>   the notification email.
> 

I could add these as permanent/hash table entries - ok?

Thanks for your feedback,
Chris

=====
Need somewhere to Live in London? - Then go to http://freeflats.com

__________________________________________________
Do You Yahoo!?
Make a great connection at Yahoo! Personals.
http://personals.yahoo.com

--
To unsubscribe, e-mail:   <mailto:jetspeed-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:jetspeed-dev-help@jakarta.apache.org>


Mime
View raw message