Michael Blow has posted comments on this change. ( https://asterix-gerrit.ics.uci.edu/2936 )
Change subject: [NO ISSUE][DASH] Asterixdb-dashboard: Plan Format| Flat Nested Objects on Table ...................................................................... Patch Set 13: (4 comments) https://asterix-gerrit.ics.uci.edu/#/c/2936/13/asterixdb/asterix-dashboard/pom.xml File asterixdb/asterix-dashboard/pom.xml: https://asterix-gerrit.ics.uci.edu/#/c/2936/13/asterixdb/asterix-dashboard/pom.xml@161 PS13, Line 161: <templateProperties> : <asterixDashboardResourcesPrefix>asterix-dashboard/src/main/resources</asterixDashboardResourcesPrefix> : </templateProperties> this shouldn't be here- it should default to dashboard/ for this context https://asterix-gerrit.ics.uci.edu/#/c/2936/13/asterixdb/asterix-dashboard/src/main/licenses/dashboard-license.ftl File asterixdb/asterix-dashboard/src/main/licenses/dashboard-license.ftl: https://asterix-gerrit.ics.uci.edu/#/c/2936/13/asterixdb/asterix-dashboard/src/main/licenses/dashboard-license.ftl@61 PS13, Line 61: dashboard looks like the other prefixes (and this one for source license context) include the trailing slash- so this should be dashboard/ Note, these single quotes should not be here, I had them in my comment to enclose the literal value. https://asterix-gerrit.ics.uci.edu/#/c/2936/13/asterixdb/asterix-dashboard/src/main/licenses/dashboard-license.ftl@72 PS13, Line 72: licenseFilePrefix it looks like the other prefixes (and this one for the source license context) include the trailing slash- so this should be: ${licenseFilePrefix}assets https://asterix-gerrit.ics.uci.edu/#/c/2936/13/asterixdb/src/main/licenses/templates/source_licenses.ftl File asterixdb/src/main/licenses/templates/source_licenses.ftl: https://asterix-gerrit.ics.uci.edu/#/c/2936/13/asterixdb/src/main/licenses/templates/source_licenses.ftl@141 PS13, Line 141: <#assign licenseComponent="AsterixDB Dashbaord"/> : <#assign licenseLocation="${asterixDashboardLocation!}"/> : <#assign licenseFilePrefix="${asterixDashboardResourcesPrefix!'dashboard'}"/> : === : ASTERIXDB Dashboard JS COMPONENTS: : : includes a number of packed subcomponents under : dashboard/static/ with separate copyright : notices and license terms. Your use of these subcomponents is subject : to the terms and condition of the following licenses. : === : : <@license licenseName="The Apache License, Version 2.0" component="AsterixDB Dashboard fonts" files=[ : "${licenseFilePrefix}/assets/fonts/material-icons/MaterialIcons-Regular.eot", : "${licenseFilePrefix}/assets/fonts/material-icons/MaterialIcons-Regular.svg", : "${licenseFilePrefix}/assets/fonts/material-icons/MaterialIcons-Regular.ttf", : "${licenseFilePrefix}/assets/fonts/material-icons/MaterialIcons-Regular.woff", : "${licenseFilePrefix}/assets/fonts/material-icons/MaterialIcons-Regular.woff2"]> : Embedded fonts in asterix-dashboard are from material-design-icons (https://github.com/google/material-design-icons) : : <@indent spaces=3> : <#list licenses as license> : <#if license.url == "http://www.apache.org/licenses/LICENSE-2.0.txt";> : ${license.content} : <#break> : </#if> : </#list> : </@indent> : </@license> I think this would be an include, not copy/pasted here, for easier maintenance -- To view, visit https://asterix-gerrit.ics.uci.edu/2936 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I7a27a5576ae21b6fda440bfc281d25e62638515c Gerrit-Change-Number: 2936 Gerrit-PatchSet: 13 Gerrit-Owner: Emilio Jose Coronado Lopez <[email protected]> Gerrit-Reviewer: Anon. E. Moose (1000171) Gerrit-Reviewer: Ian Maxon <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Michael Blow <[email protected]> Gerrit-Comment-Date: Thu, 18 Apr 2019 12:41:14 +0000 Gerrit-HasComments: Yes
