Ian Maxon 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: (5 comments) i added it to the source_licenses but im not sure where the variables in the #ifs there come from at all.. https://asterix-gerrit.ics.uci.edu/#/c/2936/11//COMMIT_MSG Commit Message: https://asterix-gerrit.ics.uci.edu/#/c/2936/11//COMMIT_MSG@7 PS11, Line 7: DASH] Asterixdb-das > should we create a component for dashboard, and use the standard here? e.g Sure, sounds like a good idea. https://asterix-gerrit.ics.uci.edu/#/c/2936/11/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/11/asterixdb/asterix-dashboard/src/main/licenses/dashboard-license.ftl@20 PS11, Line 20: <#list licenses as license> : <#if license.url == "http://www.apache.org/licenses/LICENSE-2.0.txt";> : ${license.content} : <#break> : </#if> : </#list> > use this below (should probably extract to a macro, but at least copy / pas Done https://asterix-gerrit.ics.uci.edu/#/c/2936/11/asterixdb/asterix-dashboard/src/main/licenses/dashboard-license.ftl@61 PS11, Line 61: <#assign licenseFilePrefix="${asterixDashboardResourcesPrefix!'dashboard'}"/> > this should default to 'dashboard' not empty string, since the distinct pat Done https://asterix-gerrit.ics.uci.edu/#/c/2936/11/asterixdb/asterix-dashboard/src/main/licenses/dashboard-license.ftl@72 PS11, Line 72: ${licenseFilePrefix}/assets/fonts/materi > these should be replaced with ${asterixDashboardResourcesPrefix}, which wil Done https://asterix-gerrit.ics.uci.edu/#/c/2936/11/asterixdb/asterix-dashboard/src/main/licenses/dashboard-license.ftl@81 PS11, Line 81: <#if license.url == "http://www.apache.org/licenses/LICENSE-2.0.txt";> : ${license.content} : <#break> : </#if> : </#list> : </@indent> : : </@license> : : <#include "/target/dashboard/static/3rdpartylicenses.txt"> : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : : > use logic above to avoid hardcoding the license text here Done -- 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 01:12:15 +0000 Gerrit-HasComments: Yes
