Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

naousse
Copy link
Contributor

@naousse naousse commented Jan 18, 2016

…able

Fixes #54

@max3903 max3903 added this to the 9.0 milestone Jan 18, 2016
@hbrunn
Copy link
Member

hbrunn commented Jan 18, 2016

@naousse do you have an idea what the removed code was supposed to do in the first place?

@max3903
Copy link
Member

max3903 commented Jan 18, 2016

@hbrunn We don't. It's a module from Odoo SA with not much explanation.

@hbrunn
Copy link
Member

hbrunn commented Jan 18, 2016

okay, someone/something will scream if we need it 👍

@max3903
Copy link
Member

max3903 commented Jan 22, 2016

👍

max3903 pushed a commit that referenced this pull request Jan 22, 2016
[FIX] TypeError: function lambda at 0x7f1f2cb562a8 is not JSON serializ…
@max3903 max3903 merged commit 583c493 into OCA:9.0 Jan 22, 2016
@Whytehorse
Copy link

This is still a bug in 8.0. Removing module document_page is a workaround but not a solution.

@max3903
Copy link
Member

max3903 commented Feb 9, 2016

@Whytehorse This is the pull request on version 9. Have you tested on that version as well?
Otherwise, here is the pull request for version 8: #82

@max3903
Copy link
Member

max3903 commented Feb 9, 2016

@Whytehorse I just tested on version 8 and 9 and everything works as expected. Make sure to update your code.

@Whytehorse
Copy link

So I need to update OCA Knowledge module to get the fix? Is it part of the Odoo 8.0 nightly or separate?

@max3903
Copy link
Member

max3903 commented Feb 9, 2016

@Whytehorse Yes, you need to update the knowledge module to get the fix. It is separate.

@Whytehorse
Copy link

That fixed it for me. THANK YOU!!!! I LOVE YOU!!! SMOOCH. hehe

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.

5 participants