Alon Bar-Lev has posted comments on this change.
Change subject: engine: no documentation message
......................................................................
Patch Set 1:
(2 comments)
....................................................
File backend/manager/modules/root/src/main/webapp/WEB-INF/web.xml
Line 188:
Line 189: <filter-mapping>
Line 190: <filter-name>BrandingFilter</filter-name>
Line 191: <url-pattern>/ovirt-engine</url-pattern>
Line 192: <url-pattern>/docs/*</url-pattern>
I do not understand why branding is applied to docs... the docs should be
standalone plain html that can be installed at webdav or local for review.
Line 193: <dispatcher>REQUEST</dispatcher>
Line 194: </filter-mapping>
Line 195:
Line 196: <filter-mapping>
....................................................
File packaging/branding/ovirt.brand/messages.properties
Line 66: obrand.docs.missing.appears=It appears that you do not have the {0}
documentation installed.
Line 67: obrand.docs.missing.admin=Please have your administrator install the
proper documentation.
Line 68: obrand.docs.missing.click_here=Please click here for the English
documentation.
Line 69: obrand.docs.missing.appear_once=This message will only be displayed
once per session.
Line 70: obrand.docs.missing.title=Missing language: {0}
why is this branding and not standard application messages?
--
To view, visit http://gerrit.ovirt.org/19848
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Iceae0ebf671efc951522c464db1a5b2b95dd5637
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alexander Wels <[email protected]>
Gerrit-Reviewer: Alexander Wels <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Einav Cohen <[email protected]>
Gerrit-Reviewer: Greg Sheremeta <[email protected]>
Gerrit-Reviewer: Vojtech Szocs <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches
