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

document_page porting to odoo 8.0 #24

Merged
merged 10 commits into from
Nov 16, 2014

Conversation

gborelli
Copy link
Contributor

I started from this branch - https://github.com/savoirfairelinux/knowledge/tree/8.0-document-page - to migrate document_page to oboo 8.0

I used the new API for models and wizards
and
I removed some parts that I didn't find in the original launchpad repository - http://bazaar.launchpad.net/~openerp/openobject-addons/7.0/files/head:/document_page/

@pedrobaeza
Copy link
Member

The reference you have to look up is not Launchpad, but 7.0 branch of Github: https://github.com/OCA/knowledge/tree/7.0. There are maybe some improvements done in this branch that have to be ported to 8.0. Usual method is to catch last 7.0 version of Github, and then start migrating it.

Can you check if you are treating with last version?

Thanks for the work you are making.

@gborelli
Copy link
Contributor Author

@pedrobaeza in the branch you are pointing to, there is no reference to document_page module

@pedrobaeza
Copy link
Member

Sorry, it is a deprecated Odoo module. You should see then in https://github.com/odoo/odoo/tree/7.0/addons/document_page

@gborelli
Copy link
Contributor Author

This repository - https://github.com/odoo/odoo/tree/7.0/addons/document_page - contains the same code used in this fork - https://github.com/savoirfairelinux/knowledge/tree/8.0-document-page - the baseline is the same I used for the porting.

In this commit you can see what I removed - abstract-open-solutions@3e61b4c

@max3903
Copy link
Sponsor Member

max3903 commented Nov 12, 2014

@gborelli Can you fix Travis errors ?

Thanks.

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

Coverage Status

Coverage increased (+37.52%) when pulling 01e8c80 on abstract-open-solutions:8.0-document_page into 7162d7b on OCA:8.0.

@coveralls
Copy link

Coverage Status

Coverage increased (+37.52%) when pulling 01e8c80 on abstract-open-solutions:8.0-document_page into 7162d7b on OCA:8.0.

@max3903
Copy link
Sponsor Member

max3903 commented Nov 12, 2014

👍

1 similar comment
@pedrobaeza
Copy link
Member

👍

pedrobaeza added a commit that referenced this pull request Nov 16, 2014
@pedrobaeza pedrobaeza merged commit 4576928 into OCA:8.0 Nov 16, 2014
@dreispt
Copy link
Sponsor Member

dreispt commented Nov 28, 2014

This is ready to merge.

@max3903 max3903 mentioned this pull request Apr 5, 2015
8 tasks
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

7 participants