search

MrDys / blacklight

Blacklight Plugin
http://projectblacklight.org/
Other
1 stars 1 forks source link

user_utils partial tries to show username, fails #498

Closed MrDys closed 12 years ago

MrDys commented 12 years ago

CODEBASE-313: Standard un-customized app, with devise install.

After logging in, toolbar at top shows this:

Log Out [#User:0x2b5ef4e20b48]

That link in the square brackets is to look at your user profile. It used to show your username there. Needs to be fixed, or changed to just say "Profile" or something.

MrDys commented 12 years ago

Original reporter: jrochkind

MrDys commented 12 years ago

jrochkind: committed this last week actually, as discussed above.

MrDys commented 12 years ago

cbeer: I just fixed this in my local app -- as is, the User model needs a #to_s method.

MrDys commented 12 years ago

jrochkind: Cool, makes sense. We want this to work well out of the box though, right?

So I'm thinking, we have the blacklight "--devise" generator add a #to_s after devise generation?

And we add that to our documented specs for locally implemented auths, that BL may use #to_s for a displayable username.

it's appropriate if we say we'll use #to_s for displayable usernames? I think so.