click here for details... Sakai Executive Director Position Search now open
Issue Details (XML | Word | Printable)

Key: SAK-10762
Type: Task Task
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Seth Theriault
Reporter: Seth Theriault
Votes: 1
Watchers: 7
Operations

If you were logged in you would be able to see more operations.
Sakai

Modify WS Setup to allow for adding participants via email address

Created: 16-Jul-2007 21:02   Updated: 23-Oct-2008 08:38
Component/s: Worksite Setup
Affects Version/s: 2.3.0, 2.3.1, 2.3.x, 2.4.0, 2.4.1, 2.4.x
Fix Version/s: 2.5.0

Time Tracking:
Not Specified

File Attachments: 1. Text File BSP-1010.patch.txt (2 kB)

Image Attachments:

1. AddPart.jpg
(82 kB)

2. screenshot-1.jpg
(182 kB)
Issue Links:
Duplicate
 
Relate
 

2.6.x Status: None
2.5.x Status: None
2.4.x Status: None


 Description  « Hide
Sometimes, you just can't remember a person's username but have the e-mail address. Josh Holtzman committed a change for bSpace:

BSP-1010 -- Modified WS Setup to allow for adding participants via email address.

that allows you to do this.

 All   Comments   Work Log   Change History   Subversion Commits   git Commits      Sort Order: Ascending order - Click to sort in descending order
Seth Theriault added a comment - 16-Jul-2007 21:05
Attached patch works tested against 2.4.x

John Leasia added a comment - 13-Sep-2007 10:03
Zhen can probably merge this in -
But just want to clarify. There are 2 entry boxes on the add participants page - one for account ids and one for email address based guest accounts. Does this allow an email address in the first box (currently not allowed)?

There is also a prohibition of certain domains in the 2nd box for email addresses, sakai.property controlled. So for example, with this patch, I guess someone could put in jleasia@umich.edu in the first box, which is for account names, and not be able to put jleasia@umich.edu in the second box, which is for guest accounts, which here can't have @umich.edu. I think it's a useful change, just want to see if we need to change the text on the page or something to make it clear what is going on.

Stephen Marquard added a comment - 13-Sep-2007 10:21
Example from our local version attached. We don't allow @uct.ac.za in guest emails, but I could add myself using my Sakai account email address (stephen.marquard@uct.ac.za) in the box, rather than specifying a username (as these are numeric, almost nobody knows other people's usernames offhand).

Oliver Heyer added a comment - 13-Sep-2007 11:55
This is a revision of the screen that UCB is about to implement. We have had a *lot* of confusion and problems as a result of implementing this patch without a concomitant UI change.

Stephen Marquard added a comment - 13-Sep-2007 12:55
Our users don't seem to have had difficulties with the UI.

A twist in the tale (looking at UCB's reference to official email addresses) is that in our configuration, users can change their account email address. This defeats adding someone by their official address (because they're using another one, e.g. gmail). So what we ideally want to be able to do is check against an alternate authoritative / canonical source, e.g. the email address from the system mutable profile, that the user cannot change. We haven't implemented this yet, however.

Oliver Heyer added a comment - 13-Sep-2007 15:40
It does look like UCT altered the instructional text, though. By referencing the UI, I was largely endorsing John's suggestion that the text be changed to make explicit the differences between the two boxes.

BTW, "official email address" is not entirely accurate here. You have to maintain a private ,berkeley.edu domain address (not necessarily @berkeley.edu), but you can also optionally list and expose another of your choosing. Those addresses can be used to add campus participants, too. The LDAP entry is made unique by an otherwise meaningless (to the user) numerical string, the LDAP UID. Ugh.

Zhen Qian added a comment - 14-Sep-2007 18:07
Josh's patch has been merged in r35117.

UI has also been changed according to UCB's mockup.

The demo sakai.properties file has been changed to reflect the configuration variable changes. Institutions need to adjust those variables accordingly.

Thanks,

- Zhen

Megan May added a comment - 06-Feb-2008 11:59
Tasks for 2.5.0 have been completed. We're nearing the end of the QA cycle and A lot of general regression testing has been going on for the past few months - the changes have likely been reviewed via this method of testing