Skip to content
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-81678 If Rule tab has class 'disabled' do nothing #685

Closed
wants to merge 3 commits into from

Conversation

alinedoleron
Copy link

@alinedoleron alinedoleron commented Jun 4, 2018

No description provided.

@liferay-continuous-integration
Copy link
Collaborator

To conserve resources, the PR Tester does not automatically run for every pull.

If your code changes were already tested in another pull, reference that pull in this pull so the test results can be analyzed.

If your pull was never tested, comment "ci:test" to run the PR Tester for this pull.

@alinedoleron
Copy link
Author

ci:test:relevant

@liferay-continuous-integration
Copy link
Collaborator

❌ ci:test:relevant - 26 out of 28 jobs passed in 1 hour 53 minutes 24 seconds 167 ms

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 8fe6936c72cfc0e3370095e3eadd71d1482bf624

Copied in Private Modules Branch:

Branch Name: master-private
Branch GIT ID: a99dc88ed6a8be2d03700908d6c465a1941b104a

26 out of 28 jobs PASSED
26 Successful Jobs:
For more details click here.

Failures unique to this pull:

For upstream results, click here.

Test bundle downloads:

@alinedoleron
Copy link
Author

Hi @vicnate5, please, could you check if this failure related to the fix?

@vicnate5
Copy link

vicnate5 commented Jun 5, 2018

@alinedoleron Failure not related. This test has become flaky recently. When a field is repeated, it is given a random string in the "field name". The test sometimes getting the wrong "field name" when trying to read the field name from the UI. This might be difficult to catch.

@rafaprax rafaprax closed this Jun 5, 2018
@rafaprax
Copy link
Owner

Pull request submitted to inacionery#697.

@alinedoleron alinedoleron deleted the LPS-81678 branch March 2, 2021 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants