On Fri, Aug 24, 2012 at 11:27 AM, Jasha Joachimsthal <[email protected]>wrote:
> I ran into the same issue as Raminder before with the OpenId url as > username. In case of OpenId we should think of different usernames than the > URL (in the model there is a separate field for the OpenId url, but it's > not being used). > +1 - When they first register on Rave, can we collect a username to map to that OpenID user? > > Jasha > > On 24 August 2012 17:56, Raminderjeet Singh <[email protected] > >wrote: > > > Sure i can make the change. I did not find any reference where > > /person/username was used so want with the change. > > > > Thanks > > Raminder > > > > On Aug 24, 2012, at 11:27 AM, Carlucci, Tony wrote: > > > > > Sorry for the potential duplicate email but it appears Jira's comment > > emailer is not working due to maintenance... > > > > > > Hi Raminder, I believe it's a necessity in Rave for a Person Profile > > page to be accessible via a username. Can you please revert the existing > > PersonController viewProfile function back, and create a second > controller > > endpoint which can be accessible via the id? That way it would serve both > > needs. Perhaps /person/id/{id} ? > > > > > > -1 on this commit (1376704) > > > > > > Others have comments? > > > > > > Thanks, Tony > > > > >
