Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[ADD] web_x2m_defaults_from_previous #510

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Dec 28, 2016
Merged

[ADD] web_x2m_defaults_from_previous #510

merged 1 commit into from
Dec 28, 2016

Conversation

@hbrunn
Copy link
Member

@hbrunn hbrunn commented Dec 21, 2016

x2many defaults

This module was written to allow you to use the previous line's input as defaults for the next line you add.

Usage

  • on a x2many field, say options="{'web_x2m_defaults_from_previous': ['field1', 'field2']}"
  • after the first line of input, succeeding lines will have the last input's values as default
  • demo data adds this for the ACL list field in the groups form

Known issues / Roadmap

  • many2many fields are not yet supported

@hbrunn hbrunn added this to the 8.0 milestone Dec 21, 2016
jjscarafia
jjscarafia approved these changes Dec 27, 2016
View reviewed changes
Copy link
Contributor

@jjscarafia jjscarafia left a comment

Choose a reason for hiding this comment

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

Nice feature! Tested on runbot. Clean code

moylop260
moylop260 approved these changes Dec 27, 2016
View reviewed changes
Copy link

@moylop260 moylop260 left a comment

Choose a reason for hiding this comment

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

Thank you

lasley
lasley reviewed Dec 27, 2016
View reviewed changes
Copy link
Contributor

@lasley lasley left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Neat, thanks @hbrunn

Are the booleans supposed to default as well? In the case of the demo, the access fields.

@hbrunn
Copy link
Member Author

hbrunn commented Dec 28, 2016

No, only name and model_id: https://github.com/OCA/web/pull/510/files#diff-233b2dfda416b2628574af7a0a43b7e4R9 - but if you patch one of the booleans into the view, this will work too.

lasley
lasley approved these changes Dec 28, 2016
View reviewed changes
Copy link
Contributor

@lasley lasley left a comment

Choose a reason for hiding this comment

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

Sweet thanks @hbrunn

@lasley lasley merged commit a27c39a into OCA:8.0 Dec 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@moylop260 moylop260 moylop260 approved these changes

+2 more reviewers

@jjscarafia jjscarafia jjscarafia approved these changes

@lasley lasley lasley approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

needs review

Projects

None yet

Milestone

8.0

Development

Successfully merging this pull request may close these issues.

4 participants

@hbrunn @jjscarafia @moylop260 @lasley