-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
LPS-152226 Deprecate disableToggleBoxes
and replace its usages
#2215
LPS-152226 Deprecate disableToggleBoxes
and replace its usages
#2215
Conversation
CI is automatically triggering the following test suites:
|
✔️ ci:test:sf - 1 out of 1 jobs passed in 3 minutesClick here for more details.Base Branch:Branch Name: master Sender Branch:Branch Name: disableToggleBoxes 1 Successful Jobs:For more details click here. |
disableToggleBoxe
s and replace its usagesdisableToggleBoxes
and replace its usages
Jenkins Build:test-portal-source-format#1271 Jenkins Report:jenkins-report.html Jenkins Suite:sf Pull Request:liferay-frontend#2215 Testray Routine:EE Pull Request Testray Build:[master] ci:test:sf - kresimir-coko > liferay-frontend - PR#2215 - 2022-05-10[03:45:35] Testray Importer:publish-testray-report#5072 |
Jenkins Build:test-portal-acceptance-pullrequest(master)#3080 Jenkins Report:jenkins-report.html Jenkins Suite:relevant Pull Request:liferay-frontend#2215 Testray Routine:EE Pull Request Testray Build:[master] ci:test:relevant - kresimir-coko > liferay-frontend - PR#2215 - 2022-05-10[03:45:30] Testray Importer:publish-testray-report#8191 |
modules/apps/journal/journal-web/src/main/resources/META-INF/resources/article/basic_info.jsp
Outdated
Show resolved
Hide resolved
modules/apps/journal/journal-web/src/main/resources/META-INF/resources/edit_feed.jsp
Outdated
Show resolved
Hide resolved
...et-configuration-web/src/main/resources/META-INF/resources/edit_public_render_parameters.jsp
Outdated
Show resolved
Hide resolved
...et-configuration-web/src/main/resources/META-INF/resources/edit_public_render_parameters.jsp
Outdated
Show resolved
Hide resolved
9853b09
to
93a2b6d
Compare
@bryceosterhaus I've rebased and fixed your concerns, pushing the new commit |
ci:test:sf |
ci:test:relevant |
✔️ ci:test:sf - 1 out of 1 jobs passed in 4 minutesClick here for more details.Base Branch:Branch Name: master Sender Branch:Branch Name: disableToggleBoxes 1 Successful Jobs:For more details click here. |
Jenkins Build:test-portal-source-format#1237 Jenkins Report:jenkins-report.html Jenkins Suite:sf Pull Request:liferay-frontend#2215 Testray Routine:EE Pull Request Testray Build:[master] ci:test:sf - kresimir-coko > liferay-frontend - PR#2215 - 2022-05-11[04:54:31] Testray Importer:publish-testray-report#4709 |
Jenkins Build:test-portal-acceptance-pullrequest(master)#3283 Jenkins Report:jenkins-report.html Jenkins Suite:relevant Pull Request:liferay-frontend#2215 Testray Routine:EE Pull Request Testray Build:[master] ci:test:relevant - kresimir-coko > liferay-frontend - PR#2215 - 2022-05-11[05:47:56] Testray Importer:publish-testray-report#9206 |
ci:forward |
CI is automatically triggering the following test suites:
The pull request will automatically be forwarded to the user
|
Skipping previously passed test suites: |
All required test suite(s) passed. |
Pull request has been successfully forwarded to brianchandotcom#117321 |
After careful investigation I've come to the conclusion that this utility was a more specific variant of our toggleDisabled util. I've changed usages of this util to reflect that conclusion.
🧪 Testing
I've tested only one of three usages, I couldn't find the other 2 usages. I've tested the one inside
edit_feed.jsp
that can be tested by enabling Feeds first: System settings -> Web Content -> Administration -> Show feeds. Then go to Web Content > Feeds and create a new Feed.😕 Confusion
The given example above has the elements in question set as hidden, so the functionality should be able to be removed in that example, but I wanted to play it safe and keep everything as it was. Due to this, in order to test it, you will need to replace the
type="hidden"
withtype="text"
andtype="checkbox"
, respectively (in this part).