[Freeipa-devel] webUI question

[Rob Crittenden]

Karl MacMillan wrote:
> On Tue, 2007-11-13 at 09:17 -0500, Simo Sorce wrote:
>> On Tue, 2007-11-13 at 16:06 +1000, David O'Brien wrote:
>>> Rob Crittenden wrote:
>>>> Giuseppe Paterno' wrote:
>>>>> Hi!  Quick question about the webui: why does it requires the e-mail?
>>>>> And what's the "see also" field that is required?
>>>>> A user might be not human and might not have an email associated.
>>>>> ipa-adduser does not require these fields.
>>>>> Cheers,
>>>> Some of it is just experimentation, particularly seeAlso. We want to be
>>>> able to allow users to dynamically add attributes to the new/show/edit
>>>> pages. Two are there now, seeAlso and o. As a demo one is required and
>>>> one isn't.
>>>>
>>>> As for e-mail, I'm not sure. Simo raised a similar question and we never
>>>> got it answered.
>>>>
>>>> rob
>>>>
>>> If some users are not going to be human, it makes me wonder, is "People"
>>> and "Person" the correct terminology? Ought we use "User", especially
>>> considering the underlying commands are ipa-adduser, etc.
>> Yes this is why I lobbied for cn=users and not ou=people :-)
>>
> 
> I agree with this as well - it matches existing Unix terminology and our
> commandline tools. So I think we should change to cn=users and updated
> the web gui.
> 
> Objections (other than more work)?

We already store into cn=users, just the UI need to be updated. If 
someone wants to file a trac for it I'll add it into the queue.

Along those lines, should we be requiring e-mail address for users?

I may submit a patch soon to disable the annoying required seeAlso as 
well, but it does serve as a nice reminder that there is still much work 
to be done related to loading configuration data from LDAP.

rob
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 3245 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20071113/de26b991/attachment.bin>