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
[IMP] portal: add street2 to user details page #162355
[IMP] portal: add street2 to user details page #162355
Conversation
65e2af5
to
799ebb3
Compare
Hey @khpa-odoo, LGTM 😃 Just a small change. You can update the description of the pr
|
<select name="state_id" t-attf-class="form-select #{error.get('state_id') and 'is-invalid' or ''}"> | ||
<option value="">select...</option> | ||
<t t-foreach="states or []" t-as="state"> | ||
<option t-att-value="state.id" style="display:none;" t-att-data-country_id="state.country_id.id" t-att-selected="state.id == int(state_id) if state_id else state.id == partner.state_id.id"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @khpa-odoo 😄
As we're updating those lines anyway, we could probably use the opportunity to change the ugly style="display:none;"
to a d-none
class ? 🙂
Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @amdi-odoo 😊
Sure! Updated the code 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello again 😄
d-none
isn't dynamic (no use of variables) so simply a class
instead of the t-att-class
should do it 😄
Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done : )
799ebb3
to
86078b0
Compare
This commit adds the field `Street 2` in the view of user details which is rendered by the route '/my/account' (My Account -> Addresses). It also reorganizes some fields in `portal_my_details_fields` to make them more organized. Additional: This commit replaces the style attribute used in `option` tag to set property `display: none`, with the bootstrap class `d-none`. Task-3875163
86078b0
to
49c6790
Compare
@robodoo r+ |
This commit adds the field `Street 2` in the view of user details which is rendered by the route '/my/account' (My Account -> Addresses). It also reorganizes some fields in `portal_my_details_fields` to make them more organized. Additional: This commit replaces the style attribute used in `option` tag to set property `display: none`, with the bootstrap class `d-none`. Task-3875163 closes #162355 Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
This commit adds the field `Street 2` in the view of user details which is rendered by the route '/my/account' (My Account -> Addresses). It also reorganizes some fields in `portal_my_details_fields` to make them more organized. Additional: This commit replaces the style attribute used in `option` tag to set property `display: none`, with the bootstrap class `d-none`. Task-3875163 closes #162355 Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
This commit adds the field `Street 2` in the view of user details which is rendered by the route '/my/account' (My Account -> Addresses). It also reorganizes some fields in `portal_my_details_fields` to make them more organized. Additional: This commit replaces the style attribute used in `option` tag to set property `display: none`, with the bootstrap class `d-none`. Task-3875163 closes #162355 Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
This commit adds the field `Street 2` in the view of user details which is rendered by the route '/my/account' (My Account -> Addresses). It also reorganizes some fields in `portal_my_details_fields` to make them more organized. Additional: This commit replaces the style attribute used in `option` tag to set property `display: none`, with the bootstrap class `d-none`. Task-3875163 closes odoo#162355 Signed-off-by: Thibault Delavallee (tde) <tde@openerp.com>
This PR adds the field
Street 2
in the view of user details which is rendered by the route/my/account
.(My Account -> Addresses)
It also reorganizes some fields in
portal_my_details_fields
to make them more organized as in below structure:Additional:
This PR replaces the style attribute used in
option
tag to set propertydisplay: none
, with the bootstrap classd-none
.Task-3875163