Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Login with google on startpage #32

Closed
patrickschaerer opened this issue Nov 24, 2017 · 5 comments
Closed

Login with google on startpage #32

patrickschaerer opened this issue Nov 24, 2017 · 5 comments

Comments

@patrickschaerer
Copy link

I really like the look of this theme, but I have a little issue.
On the startpage there is the login-window, which is really nice to have there. But when I try to login with google oauth2, I get the error "A required parameter (sesskey) was missing".
When I login without google it works fine. When I login with google on the login-page(mysite.com/moodle/login/index.php) it works fine as well.
How can I fix that?
or if that is not possible how can I remove the login-window on the start page?

@patrickschaerer
Copy link
Author

Ok I removed the login-box on the template frontpageguest.mustache

But mabe you could help me fix the problem because I liked the feature to have a login box on the startpage

@giselebrugger
Copy link

giselebrugger commented Nov 27, 2017

I have the same problem:
In login moove theme with facebook login link is
https://giselebrugger.com/moodle/auth/oauth2/login.php?id=2&wantsurl=%2F&sesskey=Qtk3DJVr2T

In login moodle (https://giselebrugger.com/moodle/login/index.php) with facebook login is
https://giselebrugger.com/moodle/auth/oauth2/login.php?id=2&wantsurl=https%3A%2F%2Fgiselebrugger.com%2Fmoodle%2Fauth%2Foauth2%2Flogin.php%3Fid%3D2%26amp%3Bwantsurl%3D%252F%26amp%3Bsesskey%3DQtk3DJVr2T&sesskey=Qtk3DJVr2T

see after id=2& in moove theme have : amp;
and parameter wantsurl is empty

Theme moove version: 2.0.1 2017112200
moodle version: Moodle 3.4+ (Build: 20171123)

@willianmano
Copy link
Owner

Hi @giselebrugger @patrickschaerer thannk you for share this.
I solved that issue on the last version. Please update your theme and let me know if it works.

@patrickschaerer
Copy link
Author

it works! Thanks

@giselebrugger
Copy link

Thank you a lot...
Are you The Flash? :)

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

No branches or pull requests

3 participants