Alon Bar-Lev has posted comments on this change.

Change subject: userportal, webadmin: use json for documentation mapping files
......................................................................


Patch Set 5:

(3 comments)

I believe that manual should be removed from Makefile and from spec file as 
well as we no longer install any file.

....................................................
File README.manual
Line 5: actions in the web applications.
Line 6: 
Line 7: To add context sensitive help, a few files need to be created.
Line 8: 
Line 9: 1. /usr/share/doc/DOC_PACKAGE/manual/docsconfig.properties
where is the configuration of ENGINE_MANUAL?

I think better phrasing will be something more similar to:

OVERVIEW

--------------

ovirt-engine supports manual package for context sensitive help within 
application. Manual package is not provided by base package and should be added 
externally.

PREPARATIONS

---------------------

Provided the manual package name is DOC_PACKAGE the following steps should be 
followed.

1. Prepare manual which are static html files,.

2. Create manual configuration file at docsconfig.properties within root of 
manual, create lookup index files to map between help tag and URI, see format 
bellow.

3. Package manual pages at DOC_PACKAGE to be installed into filesystem, for 
example:

 /usr/share/doc/DOC_PACKAGE/manual

4. Tell engine where manual is

 /etc/ovirt-engine/engine.conf.d/20-DOC_PACKAGE.conf

     ENGINE_MANUAL="/usr/share/doc/DOC_PACKAGE/manual"

MANUAL CONFIGURATION FILE

-----------------------------------

ovirt-engine will read the docscibfug.properties [[[[ AND I HATE THIS NAME AS 
IT IS NOT docsconfig it is index or config or manual.properties or anything but 
doc]]] located at the root of the manual content.

The format of this file is:

 helptags.APPLICATION = [index1.json[, index2.json[, ....]]]

APPLICATION is the name of the application that issue the request, the value is 
list of json files relative to configuration file that map between help tag and 
URI, they content is merged left to right, last wins.

For format of index mapping file is a dictionary key is help tag and value is 
uri relative to the package.

Example:

 {
     "create_new_vm": "newvm.html,
     "delete_vm": "delvm.html,
 }

The index file can be generated out of source code or compare to source code 
using build/doctag.py tool available within engine source respository.
Line 10: 
Line 11: The format of the properties file is:
Line 12: 
Line 13: helptags.APPLICATION = <list of files>


Line 28: Example common.json file:
Line 29: 
Line 30: {
Line 31:     "create_new_vm": "/manual/some/help/document.html",
Line 32:     "delete_vm": "/manual/another/help/document.html"
why not relative to manual?
Line 33: }
Line 34: 
Line 35: These helpTags ("create_new_vm", "delete_vm") are found in the Java 
code
Line 36: Model objects. Dialogs and other actions work with Models, so we use 
the


....................................................
File packaging/etc/engine.conf.d/30-DOC_PACKAGE.conf
Line 1: ENGINE_MANUAL="/usr/share/doc/DOC_PACKAGE/manual"
why do you added this file? oh.... I meant to document this... not to add it.


-- 
To view, visit http://gerrit.ovirt.org/21392
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4d6e8be143583d253943445842c2851fb52274e6
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Greg Sheremeta <gsher...@redhat.com>
Gerrit-Reviewer: Alexander Wels <aw...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Einav Cohen <eco...@redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gsher...@redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vsz...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to