-
-
Notifications
You must be signed in to change notification settings - Fork 152
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
[10.0][IMP]-Migrated Business requirement etherpad module from v8 to v10. #235
[10.0][IMP]-Migrated Business requirement etherpad module from v8 to v10. #235
Conversation
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.
LGTM
@seb-elico @victormartinelicocorp
@elicoidal I could review it but think would be better as soon #223 is merged so Travis, Coveralls, Runbot are operative. |
<field name="model">business.requirement</field> | ||
<field name="inherit_id" ref="business_requirement.view_business_requirement_form"/> | ||
<field name="arch" type="xml"> | ||
<xpath expr='//page[1]' position='after'> |
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.
- use
"
as attribute separator - why using absolute index instead of the tab name?
<field name="model">business.requirement</field> | ||
<field name="inherit_id" ref="business_requirement.view_business_requirement_form"/> | ||
<field name="arch" type="xml"> | ||
<xpath expr='//page[@name="logs"]' position='before'> |
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.
Use "
to encapsulate tag parameters
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.
After @seb-elico comments fixed. LGTM
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.
LGTM
…v10. (OCA#235) * [IMP]Migrated business requriement etherpad module from v8 to v10 * [IMP]Improved xpath code * [IMP]Improved " to encapsulate tag parameters
#48