Forum .LRN Q&A: Proposal to add auth::require_login to Manage memberships page

Collapse
1: Proposal to add auth::require_login to Manage memberships page
Posted by Michael Cordova on 09/08/09 01:21 PM
If a non-registered user goes to /dotlrn/manage-memberships it's being redirected to /dotlrn/not-allowed where there is only a text message.

I think it's better to use [auth::require_login] to redirect to the login page, with the return_url to /dotlrn/manage-memberships

Shall I wait for .LRN Leadership Team approval, or any other thing, before committing this change?

Collapse
2: Re: Proposal to add auth::require_login to Manage memberships page (response to 1)
Posted by Dave Bauer on 09/08/09 06:06 PM
I think this makes sense.

Redirecting to not-allowed serves no useful purpose, it doesn't tell the user anyting useful or let them do anything about it.

Collapse
3: Re: Proposal to add auth::require_login to Manage memberships page (response to 1)
Posted by Don Baccus on 09/09/09 08:35 PM
Yeah, please do it.

Put it on HEAD, please, as this isn't really a bug but an improvement, and we're already in the release process ...