Skip to content
This repository was archived by the owner on Feb 16, 2026. It is now read-only.

[FIX] Prettier login page for odoobouncer (#2314)#2

Open
KKamaa wants to merge 2 commits intomasterfrom
odoobouncer-2314
Open

[FIX] Prettier login page for odoobouncer (#2314)#2
KKamaa wants to merge 2 commits intomasterfrom
odoobouncer-2314

Conversation

@KKamaa
Copy link

@KKamaa KKamaa commented Feb 22, 2022

added sample of logins to choose from many more can be added as stand by templates for future implementations.

nginx-odoo.py Outdated
redirect_url += "/"
return self.redirect(f"{redirect_url}auth/{session_id}")
self.render(r"./templates/login.html", **config.theme_params)
self.render(r"./templates/login-2.html", **config.theme_params)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@KKamaa We currently have an environment variable for the backgoundcolor; could you create another environment variable called NGINX_ODOO_THEME or something, which defaults to login, but with which you can configure to use another theme?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@thomaspaulb added the env variable

</div>
</div>
</body>
</html> No newline at end of file
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the page for the first step, but how about the second step of entering the 2FA code? Is that also using the other theme?

Copy link
Author

@KKamaa KKamaa Feb 24, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added one template for this but support for the env variable incase future templates are added.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants