-
-
Notifications
You must be signed in to change notification settings - Fork 148
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
8.0 connector flow #17
Conversation
# There is no convention or clear specification that the return of | ||
# do_run() always is an impexp.file id. In fact I know no real | ||
# example there this is the case | ||
# |
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.
@tremlin can you say something to that?
'application': False, | ||
'images': [ | ||
], | ||
'css': [ |
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.
remove empty entries
7abbdd4
to
f11d30d
Compare
Making the relation to the task mandatory makes uninstalling tasks impossible while keeping their results.
<field name="model">impexp.chunk</field> | ||
<field name="type">form</field> | ||
<field name="arch" type="xml"> | ||
<form string="Task Form" version="7.0" create='false'> |
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.
version="7.0" is not needed anymore
connector_flow: remove deprecated view version tag
connector_flow: remove deprecated connector API
@codingforfun Can you do the remaining changes that should make this PR ok ? |
What's the most important leftovers in your opinion. |
Fix travis issue with docker-compose
Is there still interest in this? |
Add 'connector_flow' module. This module provides a minimal framework for common
data import/export tasks based on the OpenERP Connector.