-
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-131271 It's not possible to edit a field with options in the fieldset modal #196
Conversation
CI is automatically triggering the following test suites:
|
ci:stop |
✔️ 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: LPS-131271 1 Successful Jobs:For more details click here. |
Jenkins Build:test-portal-source-format#2868 Jenkins Report:jenkins-report.html Jenkins Suite:sf Pull Request:liferay-forms#196 Testray Routine:EE Pull Request Testray Importer:publish-testray-report#522 |
…dset modal - https://issues.liferay.com/browse/LPS-131271 Co-authored-by: Aline D'Oleron <aline.cantarelli@liferay.com>
ci:test:sf |
ci:test:relevant |
✔️ 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: LPS-131271 1 Successful Jobs:For more details click here. |
Jenkins Build:test-portal-source-format#5808 Jenkins Report:jenkins-report.html Jenkins Suite:sf Pull Request:liferay-forms#196 Testray Routine:EE Pull Request Testray Importer:publish-testray-report#790 |
✔️ ci:test:stable - 9 out of 9 jobs passed❌ ci:test:relevant - 24 out of 26 jobs passed in 2 hoursClick here for more details.This pull is eligible for reevaluation. When this upstream build has completed, using the following CI command will compare this pull request result against a more recent upstream result: ci:reevaluate:1137401_5877 Base Branch:Branch Name: master Upstream Comparison:Branch GIT ID: 318146d8babd408ff5f16b93b0e04428cbf3241f ci:test:stable - 9 out of 9 jobs PASSED9 Successful Jobs:
ci:test:relevant - 24 out of 26 jobs PASSED2 Failed Jobs:24 Successful Jobs:
For more details click here.Failures unique to this pull:
For upstream results, click here. |
Jenkins Build:test-portal-acceptance-pullrequest(master)#5877 Jenkins Report:jenkins-report.html Jenkins Suite:relevant Pull Request:liferay-forms#196 Testray Routine:EE Pull Request Testray Importer:publish-testray-report#611 |
✔️ ci:test:stable - 9 out of 9 jobs passed✔️ ci:test:relevant - 25 out of 26 jobs passed in 2 hours 13 minutesClick here for more details.Base Branch:Branch Name: master Upstream Comparison:Branch GIT ID: 318146d8babd408ff5f16b93b0e04428cbf3241f ci:test:stable - 9 out of 9 jobs PASSED9 Successful Jobs:
ci:test:relevant - 24 out of 26 jobs PASSED2 Failed Jobs:24 Successful Jobs:
For more details click here.This pull contains no unique failures.Failures in common with acceptance upstream results at 318146d:
|
Jenkins Build:test-portal-acceptance-pullrequest(master)#8242 Jenkins Report:jenkins-report.html Jenkins Suite:relevant Pull Request:liferay-forms#196 Testray Routine:EE Pull Request Testray Importer:publish-testray-report#669 |
case FIELD_TYPES.radio: | ||
case FIELD_TYPES.checkboxMultiple: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we remove these case, since we are not using?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually these cases are behaving the same as the case FIELD_TYPES.select
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@miltonmc, Just one question about it
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#101359 |
Ticket: LPS-131271