Skip to content

Change redirect param name to break loop#35

Open
mickaelclavreul wants to merge 1 commit into
catalyst:MOODLE_403_STABLEfrom
mickaelclavreul:fix-redirect
Open

Change redirect param name to break loop#35
mickaelclavreul wants to merge 1 commit into
catalyst:MOODLE_403_STABLEfrom
mickaelclavreul:fix-redirect

Conversation

@mickaelclavreul

Copy link
Copy Markdown

Fix for #34

@danmarsden

Copy link
Copy Markdown
Member

Thanks @mickaelclavreul - we actually need to add the "loginredirect" to the url rather than replace the "noredirect" one - there are various plugins in the wild that rely on the "noredirect" param - if you get a chance to update your patch so that it adds the extra param I'd be happy to merge this in, otherwise it might sit here for a while until someone gets a chance to add that in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants