-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[10.0] help_online fix export/import #584
Conversation
This key is required to get the pages properly imported as web pages
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.
@lmignon one minor note & can you also please raise the module up by a patch version in the manifest?
help_online/tests/common.py
Outdated
@@ -81,6 +81,7 @@ def createPage(self, pageName, imgXmlId=False): | |||
'type': 'qweb', | |||
'arch': arch, | |||
'page': True, | |||
'key': 'website.' + pageName |
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.
'website.%s' % pageName
in order to handle type conversions
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.
👍
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.
Works
This handles type conversions properly
It looks like this module is breaking the build: |
moreover, my commit history was clean and should not have been squashed. I don't want to be credited for this 💣 💥 commit. Commits from different authors can't be squashed!!!!! |
@yajo Can you fix your mistake PLZ |
This reverts commit 95f15c4.
Yes, @lmignon sorry, will fix ASAP. |
* [FIX] help_online: Replace obsolete icon into the export wizard * [FIX] help_online: Add the key attribute into the exported data This key is required to get the pages properly imported as web pages * Use the correct method for combining strings This handles type conversions properly
This reverts commit 95f15c4.
No description provided.