Skip to content

Conversation

@liferay-continuous-integration
Copy link
Collaborator

Forwarded from: liferay-frontend#5122 (Took 2 ci:forward attempts in 14 hours 41 minutes)
Console

@chestofwonders
@liferay-frontend

Original pull request comment:
Issue: https://liferay.atlassian.net/browse/LPD-24670

See liferay-frontend#5110 for previous discussion

Inform about Data Sets without visualization mode when selecting Data Set

The goal of this PR is to warn the user when selecting a Data Set from the list to display in a fragment, that there are Data Set that will not be able to be displayed on the page, since no visualization modes have been defined for them.

image

Also, a tooltip is show when user hovers the cursor over the warning icon

image

@liferay-continuous-integration
Copy link
Collaborator Author

✔️ ci:test:sf - 1 out of 1 jobs passed in 4 minutes

Ran com.liferay.source.formatter at released version 1.0.1548.

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 375f6214ece954953627aa16f77a1de631884743

Sender Branch:

Branch Name: LPD-24670
Branch GIT ID: 29c5f256151b9a914f5f61fa7a84e636f3bf94e8

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

@liferay-continuous-integration
Copy link
Collaborator Author

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

@liferay-continuous-integration
Copy link
Collaborator Author

✔️ ci:test:stable - 11 out of 11 jobs passed

❌ ci:test:relevant - 14 out of 17 jobs passed in 58 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 375f6214ece954953627aa16f77a1de631884743

Upstream Comparison:

Branch GIT ID: cb2950353b3968cfa6c10a0bd3dfebb8e8beafc3
Jenkins Build URL: EE Development Acceptance (master) - 196 - 2025-10-16[16:39:16]

ci:test:stable - 11 out of 11 jobs PASSED
11 Successful Jobs:
    ci:test:relevant - 14 out of 17 jobs PASSED

    3 Failed Jobs:

    14 Successful Jobs:
      For more details click here.

      Failures unique to this pull:

      For upstream results, click here.

      Test bundle downloads:

      @brianchandotcom
      Copy link
      Owner

      Merged. Thank you.
      View total diff: 2e4bb9d...bfc4683

      Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

      Labels

      None yet

      Projects

      None yet

      Development

      Successfully merging this pull request may close these issues.

      3 participants