Code review comment for lp:~jcsackett/launchpad/404s-confuzzle-me

Revision history for this message
j.c.sackett (jcsackett) wrote :

> Also, surely one common case of something being 404 is that you're not
> logged in _at all_ and this only changes the authenticated side of
> things.

It doesn't show in the diff, but the the unauthenticated bit is handled differently. It already points out that you're not logged in and that not being logged in may be the problem.

>
> I am not so sure about saying "log in as someone else". Do many users
> have multiple personas and log in with them interactively? Wouldn't
> those users mostly be pretty well aware of which one they're using at
> any moment?

This was specifically brought up as a point in the conversation around what to do; my understanding was it's not the most common case, but it's not precisely uncommon either. Still, I think switching the message to something like "your account may not have privileges to see this" will address the situation better than "try logging in as someone else".

« Back to merge proposal