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

base_import_async: properly register the module with connector framework #13

Merged
merged 4 commits into from Apr 21, 2015

Conversation

sbidoul
Copy link
Member

@sbidoul sbidoul commented Apr 20, 2015

Plus change README.md to README.rst

@guewen
Copy link
Member

guewen commented Apr 20, 2015

Thanks!
👍

@lmignon
Copy link
Sponsor

lmignon commented Apr 20, 2015

👍

@sebastienbeau
Copy link
Member

👍 thanks

@coveralls
Copy link

Coverage Status

Coverage increased (+0.15%) to 92.31% when pulling f4f30bb on acsone:8.0-base_import_async-install into e18bc57 on OCA:8.0.

@@ -36,14 +41,19 @@ Other modules may benefit from this infrastructure in the following way
(`header`, `encoding`, `separator`, `quoting`,
`use_connector`, `chunk_size`).

## Known limitations
Known limitations
Copy link
Member

Choose a reason for hiding this comment

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

Use better the usual section "Known issues / Roadmap"

@sbidoul
Copy link
Member Author

sbidoul commented Apr 21, 2015

@pedrobaeza I took care of your comments.

@pedrobaeza
Copy link
Member

Thanks 👍

@coveralls
Copy link

Coverage Status

Coverage increased (+0.15%) to 92.31% when pulling 31126d7 on acsone:8.0-base_import_async-install into e18bc57 on OCA:8.0.

guewen added a commit that referenced this pull request Apr 21, 2015
base_import_async: properly register the module with connector framework
@guewen guewen merged commit 1e9955a into OCA:8.0 Apr 21, 2015
@sbidoul sbidoul deleted the 8.0-base_import_async-install branch April 21, 2015 10:21
tschanzt pushed a commit to camptocamp/connector-interfaces that referenced this pull request Mar 15, 2018
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

6 participants