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

[PORT] website_login_background #131

Merged
merged 2 commits into from Nov 1, 2016

Conversation

Projects
None yet
2 participants
@iledarn
Copy link
Contributor

iledarn commented Oct 30, 2016

No description provided.

@yelizariev

This comment has been minimized.

Copy link
Member

yelizariev commented Oct 31, 2016

  • /web/signup - internal server error
  • /web/login - shows Administrator instead of Sign In
@yelizariev

This comment has been minimized.

Copy link
Member

yelizariev commented Oct 31, 2016

Probably you need to define auth attribute in website_login_background

@http.route(website=True, auth="public")
def web_login(self, redirect=None, *args, **kw):
@iledarn

This comment has been minimized.

Copy link
Contributor

iledarn commented Oct 31, 2016

@yelizariev, done

@yelizariev yelizariev merged commit 46b1241 into it-projects-llc:10.0 Nov 1, 2016

2 of 4 checks passed

ci/runbot runbot build 00901-131-1c0cc7 (runtime 466s)
Details
continuous-integration/travis-ci/pr The Travis CI build failed
Details
ci/branches Branch names are correct
Details
coverage/coveralls Coverage remained the same at 36.196%
Details

tedsalmon pushed a commit to LasLabs/website-addons that referenced this pull request Nov 3, 2016

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