Skip to content

[9.0] mig website_signup_legal_page_required - #318

Closed
eLBati wants to merge 11 commits into
OCA:9.0from
eLBati:9.0-mig-website_signup_legal_page_required
Closed

eLBati wants to merge 11 commits into
OCA:9.0from
eLBati:9.0-mig-website_signup_legal_page_required

Conversation

@eLBati

@eLBati eLBati commented Feb 3, 2017

Copy link
Copy Markdown
Member

No description provided.

yajo and others added 11 commits February 3, 2017 14:17
OCA Transbot updated translations from Transifex
It happened because the required check box was not displayed.
OCA Transbot updated translations from Transifex

OCA Transbot updated translations from Transifex

OCA Transbot updated translations from Transifex

OCA Transbot updated translations from Transifex

OCA Transbot updated translations from Transifex

OCA Transbot updated translations from Transifex

@simahawk simahawk left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

@eLBati better later than never :)

@@ -0,0 +1,7 @@
# -*- coding: utf-8 -*-
# © 2015 Antiun Ingeniería, S.L.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

pls replace (c) w/ Copyright

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

And please switch Antiun to Tecnativa, that is the new company

"summary": "Force the user to accept the legal terms to open an account",
"version": "9.0.1.0.0",
"category": "Website",
"website": "http://www.antiun.com",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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



class LegalTermsNotAcceptedError(SignupError):
def __init__(self, msg=_("You must accept our legal terms.")):

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

def __init__(self, msg=None):
    msg = msg or _("You must accept our legal terms.")

name="accepted_legal_terms"
id="accepted_legal_terms"
required="required"/>
I accept the <a href="/page/legal">legal advice</a>,

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

wrap this into a specific element so that we can override it easily, like:

<span id="legal_terms_msg">....</span>

@simahawk

simahawk commented Oct 1, 2018

Copy link
Copy Markdown

@eLBati PIIIIIIIIIIIIIING :)

@simahawk

simahawk commented Oct 1, 2018

Copy link
Copy Markdown

OCA CS today: @eLBati said is not going to finish this and we have it already on v10.
Feel free to re-open if needed :)

@simahawk simahawk closed this Oct 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants