Skip to content

Conversation

@ivantodorovich
Copy link
Contributor

@ivantodorovich ivantodorovich commented Apr 13, 2018

Mig #155 to 10.0 (1/2)

Depends on #160

@pedrobaeza
Copy link
Member

The module is already migrated, so this is an improvement.

@pedrobaeza pedrobaeza changed the title [MIG][10.0][document_page] Change Requests and workflow improvements on documents (#155) [10.0][IMP] document_page: Change Requests and workflow improvements on documents (#155) Apr 13, 2018
@ivantodorovich
Copy link
Contributor Author

Well, yeah, it's the migration of an improvement pushed to 9.0.
Should I change the commit message?

@pedrobaeza
Copy link
Member

Yeah, it's better to indicate correctly when seeing the commit history for this module for not seeing 2 times "migration to 10.0"

@ivantodorovich
Copy link
Contributor Author

@max3903 these are ready. Runbot and travis are failing because they need each other.

@ivantodorovich
Copy link
Contributor Author

ivantodorovich commented Apr 13, 2018 via email

@max3903
Copy link
Member

max3903 commented Apr 13, 2018

@ivantodorovich Yep, I understood this after writing the message. That's why I deleted it. Thanks!

@ivantodorovich
Copy link
Contributor Author

document_page tests are ok now https://travis-ci.org/OCA/knowledge/jobs/366307885#L827

Copy link
Member

@dreispt dreispt left a comment

Choose a reason for hiding this comment

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

LGTM

@max3903 max3903 merged commit fd07c11 into OCA:10.0 Apr 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants