[bealdav]

• Improve report esthetic
• Better way to complete doc: dedicate view on action window
• Allow to display / print the doc for a complete base menu (Sales, Purchase, etc)
• Provide a mechanism to update partially advanced_help and advanced_help_model fields: any other modules should can add or update some parts of these fields (data of several module in one fields)
• Provide a mechanism to delete properly help data at uninstall of the module (several modules have modified help fields)
• add test unit

---

When you click on 'Sales help' above, you open the report below (all documentation of the menu sales).

---

Here is a dedicate view on action window to modify end users help fields

[dreispt]

This is a new dependency, right? Isn't LXML suitable for the job?

[dreispt]

I'm not trying to push anything on you, and only as a reminder it would be desirable that help_popup and help_online eventually converge into a single module.

[bealdav]

@dreispt why not, have you a plan in mind about functionalities that should stay or be extended and with which dependencies ?
cc @lmignon @Cedric-Pigeon

[rafaelbn]

Hi, I have been reviewing this module with @bealdav and it looks very simple and useful. About @dreispt comment #417 (comment), I agree that it's better not to depend in website as help_online do: https://github.com/OCA/web/blob/9.0/help_online/__openerp__.py#L30

This module is very interesting as potentially could solve the problem of Odoo with a full functional documentation made by end-users. In this moment the doc which end user write down is stored in Odoo database so if I do the doc I must export somehow to import for my other customers.

I will wait to have travis and runbot green to final review.

[petrus-v]

Could you remove or fix comment code?

[petrus-v]

I guess this could be at least a warning error, if we want to install this module I guess this import is required

[bealdav]

this logger is for not installed module (oca guidelines)
There is raise in standard odoo installation module with 'external_dependencies' data in manifest.
No way to install if no dependency satisfied.

[gurneyalex]

please fix the tests: https://travis-ci.org/OCA/web/jobs/164951115#L544

[github-actions]

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

[bealdav]

Help_popup revinvented OCA/knowledge#653