Skip to content

Commit

Permalink
LPS-198165 Remove Release FF for import maps and Theme SVG client ext…
Browse files Browse the repository at this point in the history
…ensions
  • Loading branch information
chestofwonders authored and brianchandotcom committed Oct 21, 2023
1 parent e8f6017 commit 3fcfdce
Show file tree
Hide file tree
Showing 20 changed files with 21 additions and 92 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -56,16 +56,6 @@ public void validate(
ClientExtensionEntryConstants.TYPE_FDS_FILTER, "LPS-164563"
).put(

// feature.flag.LPS-166479

ClientExtensionEntryConstants.TYPE_THEME_SPRITEMAP, "LPS-166479"
).put(

// feature.flag.LPS-172903

ClientExtensionEntryConstants.TYPE_JS_IMPORT_MAPS_ENTRY, "LPS-172903"
).put(

// feature.flag.LPS-177027

ClientExtensionEntryConstants.TYPE_STATIC_CONTENT, "LPS-177027"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,18 +35,16 @@ LayoutLookAndFeelDisplayContext layoutLookAndFeelDisplayContext = new LayoutLook
/>
</liferay-frontend:fieldset>

<c:if test='<%= FeatureFlagManagerUtil.isEnabled("LPS-166479") %>'>
<liferay-frontend:fieldset
collapsed="<%= false %>"
collapsible="<%= true %>"
label="theme-spritemap-client-extension"
>
<react:component
module="js/layout/look_and_feel/ThemeSpritemapCETsConfiguration"
props="<%= layoutLookAndFeelDisplayContext.getThemeSpritemapCETConfigurationProps(Layout.class.getName(), selLayout.getPlid()) %>"
/>
</liferay-frontend:fieldset>
</c:if>
<liferay-frontend:fieldset
collapsed="<%= false %>"
collapsible="<%= true %>"
label="theme-spritemap-client-extension"
>
<react:component
module="js/layout/look_and_feel/ThemeSpritemapCETsConfiguration"
props="<%= layoutLookAndFeelDisplayContext.getThemeSpritemapCETConfigurationProps(Layout.class.getName(), selLayout.getPlid()) %>"
/>
</liferay-frontend:fieldset>

<liferay-frontend:fieldset
collapsed="<%= false %>"
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -35,18 +35,16 @@ LayoutSet selLayoutSet = layoutsAdminDisplayContext.getSelLayoutSet();
/>
</liferay-frontend:fieldset>

<c:if test='<%= FeatureFlagManagerUtil.isEnabled("LPS-166479") %>'>
<liferay-frontend:fieldset
collapsed="<%= false %>"
collapsible="<%= true %>"
label="theme-spritemap-client-extension"
>
<react:component
module="js/layout/look_and_feel/ThemeSpritemapCETsConfiguration"
props="<%= layoutLookAndFeelDisplayContext.getThemeSpritemapCETConfigurationProps(LayoutSet.class.getName(), selLayoutSet.getLayoutSetId()) %>"
/>
</liferay-frontend:fieldset>
</c:if>
<liferay-frontend:fieldset
collapsed="<%= false %>"
collapsible="<%= true %>"
label="theme-spritemap-client-extension"
>
<react:component
module="js/layout/look_and_feel/ThemeSpritemapCETsConfiguration"
props="<%= layoutLookAndFeelDisplayContext.getThemeSpritemapCETConfigurationProps(LayoutSet.class.getName(), selLayoutSet.getLayoutSetId()) %>"
/>
</liferay-frontend:fieldset>

<liferay-frontend:fieldset
collapsed="<%= false %>"
Expand Down
40 changes: 0 additions & 40 deletions portal-impl/src/portal.properties
Original file line number Diff line number Diff line change
Expand Up @@ -6488,11 +6488,6 @@
#
feature.flag.LPS-166126=false

#
# Env: LIFERAY_FEATURE_PERIOD_FLAG_PERIOD__UPPERCASEL__UPPERCASEP__UPPERCASES__MINUS__NUMBER1__NUMBER6__NUMBER6__NUMBER4__NUMBER7__NUMBER9_
#
feature.flag.LPS-166479=false

#
# Env: LIFERAY_FEATURE_PERIOD_FLAG_PERIOD__UPPERCASEL__UPPERCASEP__UPPERCASES__MINUS__NUMBER1__NUMBER6__NUMBER9__NUMBER8__NUMBER3__NUMBER7_
#
Expand Down Expand Up @@ -6528,11 +6523,6 @@
#
feature.flag.LPS-172416=false

#
# Env: LIFERAY_FEATURE_PERIOD_FLAG_PERIOD__UPPERCASEL__UPPERCASEP__UPPERCASES__MINUS__NUMBER1__NUMBER7__NUMBER2__NUMBER9__NUMBER0__NUMBER3_
#
feature.flag.LPS-172903=false

#
# Env: LIFERAY_FEATURE_PERIOD_FLAG_PERIOD__UPPERCASEL__UPPERCASEP__UPPERCASES__MINUS__NUMBER1__NUMBER7__NUMBER3__NUMBER1__NUMBER3__NUMBER5_
#
Expand Down Expand Up @@ -6912,16 +6902,6 @@
#
feature.flag.LPS-148856.title=Object Model Builder

#
# Env: LIFERAY_FEATURE_PERIOD_FLAG_PERIOD__UPPERCASEL__UPPERCASEP__UPPERCASES__MINUS__NUMBER1__NUMBER6__NUMBER6__NUMBER4__NUMBER7__NUMBER9__PERIOD_DESCRIPTION
#
feature.flag.LPS-166479.description=Developers will able to use theme spritemap client extensions.

#
# Env: LIFERAY_FEATURE_PERIOD_FLAG_PERIOD__UPPERCASEL__UPPERCASEP__UPPERCASES__MINUS__NUMBER1__NUMBER6__NUMBER6__NUMBER4__NUMBER7__NUMBER9__PERIOD_TYPE
#
feature.flag.LPS-166479.type=release

#
# Env: LIFERAY_FEATURE_PERIOD_FLAG_PERIOD__UPPERCASEL__UPPERCASEP__UPPERCASES__MINUS__NUMBER1__NUMBER6__NUMBER4__NUMBER5__NUMBER6__NUMBER3__PERIOD_TITLE
#
Expand All @@ -6937,11 +6917,6 @@
#
feature.flag.LPS-164563.type=beta

#
# Env: LIFERAY_FEATURE_PERIOD_FLAG_PERIOD__UPPERCASEL__UPPERCASEP__UPPERCASES__MINUS__NUMBER1__NUMBER6__NUMBER6__NUMBER4__NUMBER7__NUMBER9__PERIOD_TITLE
#
feature.flag.LPS-166479.title=Theme Spritemap Client Extension

#
# Env: LIFERAY_FEATURE_PERIOD_FLAG_PERIOD__UPPERCASEL__UPPERCASEP__UPPERCASES__MINUS__NUMBER1__NUMBER7__NUMBER1__NUMBER3__NUMBER6__NUMBER4__PERIOD_DESCRIPTION
#
Expand All @@ -6957,21 +6932,6 @@
#
feature.flag.LPS-171364.type=beta

#
# Env: LIFERAY_FEATURE_PERIOD_FLAG_PERIOD__UPPERCASEL__UPPERCASEP__UPPERCASES__MINUS__NUMBER1__NUMBER7__NUMBER2__NUMBER9__NUMBER0__NUMBER3__PERIOD_DESCRIPTION
#
feature.flag.LPS-172903.description=Developers will able to use JS import maps entry client extensions.

#
# Env: LIFERAY_FEATURE_PERIOD_FLAG_PERIOD__UPPERCASEL__UPPERCASEP__UPPERCASES__MINUS__NUMBER1__NUMBER7__NUMBER2__NUMBER9__NUMBER0__NUMBER3__PERIOD_TYPE
#
feature.flag.LPS-172903.type=release

#
# Env: LIFERAY_FEATURE_PERIOD_FLAG_PERIOD__UPPERCASEL__UPPERCASEP__UPPERCASES__MINUS__NUMBER1__NUMBER7__NUMBER2__NUMBER9__NUMBER0__NUMBER3__PERIOD_TITLE
#
feature.flag.LPS-172903.title=JS Import Maps Entry Client Extension

#
# Env: LIFERAY_FEATURE_PERIOD_FLAG_PERIOD__UPPERCASEL__UPPERCASEP__UPPERCASES__MINUS__NUMBER1__NUMBER7__NUMBER4__NUMBER4__NUMBER1__NUMBER7__PERIOD_DESCRIPTION
#
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1197,8 +1197,6 @@ definition {
@description = "LPS-166479. Created (theme SVG extensions), the theme SVG extensions admin page lists the name and type"
@priority = 4
test ThemeSvgTypeCanBeCreated {
property custom.properties = "feature.flag.LPS-166479=true";

task ("Given a created Client Extension") {
ClientExtensionGeneral.goToRemoteAppsPortlet();
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,6 @@ definition {
@description = "LPS-166479. Verify that new SVG spritemap is applied on the page"
@priority = 3
test CanExtendSvgFileFromDM {
property custom.properties = "feature.flag.LPS-166479=true";
property portal.acceptance = "true";

var portalURL = PropsUtil.get("portal.url");
Expand Down Expand Up @@ -82,7 +81,6 @@ definition {
@description = "LPS-173318. Verify that new SVG spritemap file from external site is applied on the page"
@priority = 3
test CanExtendSvgFileFromExternalSite {
property custom.properties = "feature.flag.LPS-166479=true";
property portal.acceptance = "true";

task ("A created Theme svg extensions using the external URL") {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
@component-name = "liferay-sample-workspace"
definition {

property custom.properties = "feature.flag.LPS-166479=true";
property portal.release = "true";
property portal.upstream = "true";
property test.liferay.virtual.instance = "false";
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
@component-name = "liferay-sample-workspace"
definition {

property custom.properties = "feature.flag.LPS-166479=true";
property portal.release = "true";
property portal.upstream = "true";
property test.liferay.virtual.instance = "true";
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,5 @@
#jdbc.default.password=

feature.flag.LPS-164563=true
feature.flag.LPS-166479=true
feature.flag.LPS-172903=true
feature.flag.LPS-177027=true
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,5 @@
#jdbc.default.password=

feature.flag.LPS-164563=true
feature.flag.LPS-166479=true
feature.flag.LPS-172903=true
feature.flag.LPS-177027=true
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,5 @@
#jdbc.default.password=

feature.flag.LPS-164563=true
feature.flag.LPS-166479=true
feature.flag.LPS-172903=true
feature.flag.LPS-177027=true
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,5 @@
#jdbc.default.password=

feature.flag.LPS-164563=true
feature.flag.LPS-166479=true
feature.flag.LPS-172903=true
feature.flag.LPS-177027=true
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,5 @@
#jdbc.default.password=

feature.flag.LPS-164563=true
feature.flag.LPS-166479=true
feature.flag.LPS-172903=true
feature.flag.LPS-177027=true
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,5 @@
#jdbc.default.password=

feature.flag.LPS-164563=true
feature.flag.LPS-166479=true
feature.flag.LPS-172903=true
feature.flag.LPS-177027=true
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,5 @@
#jdbc.default.password=

feature.flag.LPS-164563=true
feature.flag.LPS-166479=true
feature.flag.LPS-172903=true
feature.flag.LPS-177027=true
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,5 @@
#jdbc.default.password=

feature.flag.LPS-164563=true
feature.flag.LPS-166479=true
feature.flag.LPS-172903=true
feature.flag.LPS-177027=true
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,5 @@
#jdbc.default.password=

feature.flag.LPS-164563=true
feature.flag.LPS-166479=true
feature.flag.LPS-172903=true
feature.flag.LPS-177027=true
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,5 @@
#jdbc.default.password=

feature.flag.LPS-164563=true
feature.flag.LPS-166479=true
feature.flag.LPS-172903=true
feature.flag.LPS-177027=true
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,5 @@
#jdbc.default.password=

feature.flag.LPS-164563=true
feature.flag.LPS-166479=true
feature.flag.LPS-172903=true
feature.flag.LPS-177027=true
2 changes: 1 addition & 1 deletion workspaces/refresh_liferay_sample_workspace.sh
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@ function refresh_liferay_sample_workspace {

echo -en "\n**/dist\n**/node_modules_cache\n.DS_Store" >> .gitignore

echo -en "\n\nfeature.flag.LPS-166479=true\nfeature.flag.LPS-172903=true\nfeature.flag.LPS-164563=true\nfeature.flag.LPS-177027=true" >> configs/local/portal-ext.properties
echo -en "\n\nfeature.flag.LPS-172903=true\nfeature.flag.LPS-164563=true\nfeature.flag.LPS-177027=true" >> configs/local/portal-ext.properties

#echo -en "\nliferay.workspace.docker.image.liferay=liferay/dxp:7.4.13-u54-d5.0.5-20221208173455" >> gradle.properties
echo -en "\nliferay.workspace.node.package.manager=yarn" >> gradle.properties
Expand Down

0 comments on commit 3fcfdce

Please sign in to comment.