Yaniv Dary has posted comments on this change.

Change subject: core : Welcome page should use base reports URL
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.ovirt.org/#/c/33493/1/backend/manager/modules/services/src/main/webapp/WEB-INF/web.xml
File backend/manager/modules/services/src/main/webapp/WEB-INF/web.xml:

Line 64:       </init-param>
Line 65:   </servlet>
Line 66:   <servlet-mapping>
Line 67:       <servlet-name>reports-base-redirect</servlet-name>
Line 68:       <url-pattern>/reports-base-redirect</url-pattern>
This should be reports-redirect
Line 69:   </servlet-mapping>
Line 70: 
Line 71:   <servlet>
Line 72:       <servlet-name>reports-redirect</servlet-name>


Line 76:           <param-value>%{ENGINE_REPORTS_DASHBOARD_URL}</param-value>
Line 77:       </init-param>
Line 78:   </servlet>
Line 79:   <servlet-mapping>
Line 80:       <servlet-name>reports-redirect</servlet-name>
This should be reports-dashboard-redirect
Line 81:       <url-pattern>/reports-redirect</url-pattern>
Line 82:   </servlet-mapping>
Line 83: 
Line 84:   <servlet>


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I732df3a7d6ebec5915be21bb945889d8660c615c
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com>
Gerrit-Reviewer: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Yaniv Dary <yd...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to