Skip to content
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

[REF] 8.0 document page procedure #35

Merged
merged 7 commits into from
Apr 10, 2015

Conversation

llacroix
Copy link
Contributor

Marked as installable no other change required

Depends on

Edit add dependencies (yvaucher)

@llacroix llacroix changed the title 8.0 document page procedure [REF] 8.0 document page procedure Nov 27, 2014
@@ -30,7 +30,7 @@
"depends": ['document_page_work_instructions'],
"data": ['document_page_procedure.xml'],
"demo": [],
'installable': False,
'installable': True,
"certificate": ''
}
# vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4:

Choose a reason for hiding this comment

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

Please remove #vim lines

@bwrsandman
Copy link

Please split document_page_procedure.xml into document_page_data.xml and document_page_view.xml

@bwrsandman bwrsandman added this to the 8.0 milestone Nov 27, 2014
@bwrsandman
Copy link

Depends on #26

@llacroix
Copy link
Contributor Author

Splitted and moved icon

@max3903 max3903 removed the 8.0 label Feb 14, 2015
@max3903 max3903 mentioned this pull request Feb 14, 2015
26 tasks
@llacroix llacroix force-pushed the 8.0-document_page_procedure branch from b554e8b to d0bd88c Compare April 1, 2015 12:29
@@ -0,0 +1,43 @@
<?xml version="1.0" encoding="UTF-8"?>
<openerp>
<data>
Copy link
Sponsor Member

Choose a reason for hiding this comment

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

nopudate="1"?

@dreispt
Copy link
Sponsor Member

dreispt commented Apr 7, 2015

Minor notes, but I'm 👍

</field>
</record>

<record id="document_page_procedure" model="document.page">
Copy link
Sponsor Member

Choose a reason for hiding this comment

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

This record can be removed or moved to a demo data file.

@dreispt
Copy link
Sponsor Member

dreispt commented Apr 10, 2015

Needs fixing, based on @max3903 comment.

@yvaucher
Copy link
Member

Dependencies are merged but need fixing

@llacroix llacroix force-pushed the 8.0-document_page_procedure branch from e7c07b0 to 3c70424 Compare April 10, 2015 17:59
@llacroix
Copy link
Contributor Author

All fixed push

@max3903
Copy link
Sponsor Member

max3903 commented Apr 10, 2015

Tested on runbot

👍

max3903 pushed a commit that referenced this pull request Apr 10, 2015
@max3903 max3903 merged commit 1e7e0e4 into OCA:8.0 Apr 10, 2015
@max3903 max3903 deleted the 8.0-document_page_procedure branch April 10, 2015 20:56
tschanzt pushed a commit to camptocamp/management-system that referenced this pull request Jun 26, 2019
BSCOS-152 install document_page_quality_manual
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants