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-173179 Create UI for start of new FDS View creation #3012

Closed
wants to merge 4 commits into from

Conversation

markocikos
Copy link
Collaborator

References

What is the goal of this PR?

In this PR:

  • Main task is to create UI for start of new FDS View creation. This includes empty FDS and new dataset modal.
  • We are extending FDS Creation Menu API to support onClick on items. Without it, we would be limited to FDS-specific and limited modal. With it, we can use openModal and customize it as we like.
  • We are improving accessibility on all related buttons, fixing 7 warning errors in console.

Notes:

  • Provider values are placeholders, to test autocomplete and prepare data wiring. We will implement display of real headless resources in a separate task.
  • We are making a deliberate choice to implement FDS directly, as a React component. This is in contrast to standard JSP tag implementation in DXP. Intent is to iron out quirks and improve developer experience with this type of development. A sample of this is addition of onClick to creation button in this PR

What does it look like?

screen.mp4

Steps to reproduce

  1. Add feature.flag.LPS-164563=true to portal-ext.properties
  2. Open Global Menu > Control Panel > Object > Datasets

@liferay-continuous-integration
Copy link
Collaborator

CI is automatically triggering the following test suites:

  •     ci:test:relevant
  •     ci:test:sf

@liferay-continuous-integration
Copy link
Collaborator

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

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 569300c7313c5ce484a3b6d2868dc36520149c7e

Sender Branch:

Branch Name: LPS-173179
Branch GIT ID: 0eef0d0e898503825e92f221b5fdebd273fe8322

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

@liferay-continuous-integration
Copy link
Collaborator

@liferay-continuous-integration
Copy link
Collaborator

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

❌ ci:test:relevant - 62 out of 69 jobs passed in 1 hour 24 minutes

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 569300c7313c5ce484a3b6d2868dc36520149c7e

Upstream Comparison:

Branch GIT ID: 569300c7313c5ce484a3b6d2868dc36520149c7e
Jenkins Build URL: EE Development Acceptance (master) - 4238 - 2023-01-29[22:50:26]

ci:test:stable - 27 out of 27 jobs PASSED
27 Successful Jobs:
    ci:test:relevant - 62 out of 69 jobs PASSED

    7 Failed Jobs:

    62 Successful Jobs:
      For more details click here.

      Failures unique to this pull:


      Failures in common with acceptance upstream results at 569300c:
      1. ...
      Test bundle downloads:

      @liferay-continuous-integration
      Copy link
      Collaborator

      @markocikos
      Copy link
      Collaborator Author

      ci:test:relevant

      @liferay-continuous-integration
      Copy link
      Collaborator

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

      ❌ ci:test:relevant - 65 out of 69 jobs passed in 1 hour 43 minutes

      Click 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:1387401_6063

      Base Branch:

      Branch Name: master
      Branch GIT ID: 324a611760a954902a602c3ac0737f1a9e9a42c4

      Upstream Comparison:

      Branch GIT ID: 3073d0a3d8084b7d42545a2f024ca00f6295bfbd
      Jenkins Build URL: EE Development Acceptance (master) - 4242 - 2023-01-30[20:13:53]

      ci:test:stable - 27 out of 27 jobs PASSED
      27 Successful Jobs:
        ci:test:relevant - 65 out of 69 jobs PASSED

        4 Failed Jobs:

        65 Successful Jobs:
          For more details click here.

          Failures unique to this pull:


          Failures in common with acceptance upstream results at 3073d0a:
          Test bundle downloads:

          @liferay-continuous-integration
          Copy link
          Collaborator

          @markocikos
          Copy link
          Collaborator Author

          ci:test:relevant

          @liferay-continuous-integration
          Copy link
          Collaborator

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

          Click here for more details.

          Base Branch:

          Branch Name: master
          Branch GIT ID: fe66b60dfb479e4bd8428c8bf9eb06d16116a157

          Sender Branch:

          Branch Name: LPS-173179
          Branch GIT ID: c4dbd560b5718d588abc604c47afdc93f1087e8a

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

          @liferay-continuous-integration
          Copy link
          Collaborator

          All required test suite(s) passed.
          Forwarding pull request to brianchandotcom.
          Console

          @liferay-continuous-integration
          Copy link
          Collaborator

          Unexpected errors occurred while attempting to forward pull request to 'brianchandotcom'. Retrying in 1 minute...
          Console

          2 similar comments
          @liferay-continuous-integration
          Copy link
          Collaborator

          Unexpected errors occurred while attempting to forward pull request to 'brianchandotcom'. Retrying in 1 minute...
          Console

          @liferay-continuous-integration
          Copy link
          Collaborator

          Unexpected errors occurred while attempting to forward pull request to 'brianchandotcom'. Retrying in 1 minute...
          Console

          @markocikos
          Copy link
          Collaborator Author

          ci:forward

          @liferay-continuous-integration
          Copy link
          Collaborator

          CI is automatically triggering the following test suites:

          •     ci:test:relevant
          •     ci:test:sf

          The pull request will automatically be forwarded to the user brianchandotcom if the following test suites pass:

          •     ci:test:relevant
          •     ci:test:sf
          •     ci:test:stable

          @liferay-continuous-integration
          Copy link
          Collaborator

          Skipping previously passed test suites:
          ci:test:relevant
          ci:test:sf

          @liferay-continuous-integration
          Copy link
          Collaborator

          All required test suite(s) passed.
          Forwarding pull request to brianchandotcom.
          Console

          @liferay-continuous-integration
          Copy link
          Collaborator

          Pull request has been successfully forwarded to brianchandotcom#129563
          Console

          @liferay-continuous-integration
          Copy link
          Collaborator

          @john-co
          Copy link

          john-co commented Feb 1, 2023

          ci:test:objects

          @liferay-continuous-integration
          Copy link
          Collaborator

          The test suite specified is not a valid test suite.
          Please refer to the available test suites below.

          ci:test[:suite][:SHA|nocompile|norebase|gist-SHA]

              Test the pull request. Optionally specify the name of the test suite and/or optionally specify the upstream SHA to test against or use "nocompile" to test with a prebuilt bundle or use "norebase" to test without rebasing.

          Available test suites:
          •     ci:test:acceptance - Test acceptance tests.
          •     ci:test:acceptance-functional - Test acceptance functional tests.
          •     ci:test:acceptance-functional-db-partition - Test acceptance functional tests with Database Partitioning.
          •     ci:test:acceptance-release - Compile a release bundle and execute acceptance tests.
          •     ci:test:acceptance-upstream - Test acceptance upstream tests.
          •     ci:test:account - Test Account tests.
          •     ci:test:analytics-cloud - Test Analytics Cloud tests.
          •     ci:test:analytics-cloud-acceptance - No description is available.
          •     ci:test:analytics-cloud-quarantine - No description is available.
          •     ci:test:app-servers - Test Deployments on App Servers.
          •     ci:test:auto-tagging - Test Auto Tagging tests.
          •     ci:test:backend - Test backend tests.
          •     ci:test:backend-hypersonic - Test backend tests on Hypersonic.
          •     ci:test:blade - Test Blade samples tests.
          •     ci:test:blogs - Test Blogs tests.
          •     ci:test:bulk-editing - Test Bulk Editing tests.
          •     ci:test:bundle - Saves a compiled tomcat bundle.
          •     ci:test:calendar - Test Calendar tests.
          •     ci:test:click-to-chat - Test Click to Chat tests.
          •     ci:test:clustering - Test clustering tests.
          •     ci:test:comment - Test Comment tests.
          •     ci:test:commerce - Test only Commerce tests.
          •     ci:test:commerce-database - Test only Commerce Database tests.
          •     ci:test:commerce-es7-remote - Test only Commerce tests with the latest compatible remote Elasticsearch 7 version.
          •     ci:test:commerce-functional - Test only Commerce functional tests.
          •     ci:test:content-page-review - Test Content Page Review (comments) tests.
          •     ci:test:continuous-integration - Test Continuous Integration tests.
          •     ci:test:cookies - Test Cookie Preferences tests.
          •     ci:test:data-engine - Test Data Engine functional tests.
          •     ci:test:data-integration - Test only Data Integration tests.
          •     ci:test:data-integration-functional - Test only functional Data Integration tests.
          •     ci:test:db-partition - Test Database Partitioning functional tests.
          •     ci:test:depot - Test Depot tests.
          •     ci:test:digital-signature - Test Digital Signature tests.
          •     ci:test:dl-store - Test Document Library Store tests.
          •     ci:test:document - Test Document and Media tests.
          •     ci:test:drools - Test Drools tests.
          •     ci:test:dummy - No description is available.
          •     ci:test:echo - Test all Echo (WCM) team tests.
          •     ci:test:echo-acceptance - Test Echo (WCM) team acceptance tests.
          •     ci:test:echo-db-partition - Test Echo (WCM) team acceptance tests with Database Partitioning.
          •     ci:test:echo-headless - Test Echo (WCM) team headless components.
          •     ci:test:echo-upgrade - Test Echo (WCM) team upgrade tests.
          •     ci:test:environments - Test environment acceptance functional tests on all available environment stacks.
          •     ci:test:export-import - Test exportimport tests.
          •     ci:test:forms - Test forms tests.
          •     ci:test:forms-functional - Test only functional forms tests.
          •     ci:test:friendly-url - Test Friendly URL tests.
          •     ci:test:frontend - Test all Frontend Infra team tests.
          •     ci:test:frontend-clay - Test Frontend Infra team Clay tests.
          •     ci:test:frontend-client-extensions - Test frontend client extensions tests.
          •     ci:test:frontend-data-set - Test frontend data set tests.
          •     ci:test:frontend-dev-exp - Test frontend developer experience tests.
          •     ci:test:frontend-ui - Test frontend ui tests.
          •     ci:test:frontend-walkthrough - Test frontend walkthrough tests.
          •     ci:test:gauntlet - Test upstream tests.
          •     ci:test:gauntlet-bucket - Test a subset of upstream tests.
          •     ci:test:gradle-plugins - Test Gradle Plugins tests.
          •     ci:test:headless - Test all headless team tests.
          •     ci:test:headless-acceptance - Test headless team acceptance tests considering Headless and Rest Builder.
          •     ci:test:headless-functional - Headless Team functional API tests.
          •     ci:test:item-selector - Test item selector tests.
          •     ci:test:jdk11 - Test acceptance, smoke, and upgrade functional tests against JDK11.
          •     ci:test:jsf - Test Liferay Faces tests.
          •     ci:test:kaleo-forms - Test Kaleo Forms admin tests.
          •     ci:test:knowledge-base - No description is available.
          •     ci:test:ldap - Test LDAP tests.
          •     ci:test:license - Compile a release bundle and execute tests for licensing.
          •     ci:test:license-cluster - Compile a release bundle and execute tests for cluster licensing on 32GB slaves.
          •     ci:test:liferay-online - Compile a release bundle and execute tests with liferay online properties.
          •     ci:test:lima - Test Lima tests.
          •     ci:test:lima-acceptance - Test Lima acceptance tests.
          •     ci:test:lima-db-partition - Test Lima functional tests with Database Partitioning.
          •     ci:test:lima-upgrade - Test Lima upgrade tests.
          •     ci:test:lpkg - Test LPKG tests.
          •     ci:test:message-boards - No description is available.
          •     ci:test:mfa - Test Multi Factor Authentication tests.
          •     ci:test:notifications - Test Notifications tests.
          •     ci:test:oauth2 - Test OAuth 2 tests.
          •     ci:test:object - Test Object tests.
          •     ci:test:oidc - Test OpenID Connect tests.
          •     ci:test:onedrive - Test OneDrive tests.
          •     ci:test:online-editing - Test Online Editing tests.
          •     ci:test:osb-site-initializer-customer-portal - No description is available.
          •     ci:test:osb-site-initializer-partner-portal - No description is available.
          •     ci:test:password-policies - Test Password Policies tests.
          •     ci:test:portlet-deployment - Test deployment of all demo portlet plugins.
          •     ci:test:poshi - No description is available.
          •     ci:test:poshi-gauntlet - No description is available.
          •     ci:test:project-templates - Test project-templates tests.
          •     ci:test:publications - Test publications tests.
          •     ci:test:publications-db-partition - Test publications functional tests with Database Partitioning.
          •     ci:test:publications-environments - No description is available.
          •     ci:test:questions - No description is available.
          •     ci:test:redirect - Test Redirection tests.
          •     ci:test:relevant - Test relevant changes only.
          •     ci:test:reports - Test Reports tests.
          •     ci:test:roles - Test Roles tests.
          •     ci:test:saml - Test SAML tests.
          •     ci:test:search - Test search tests with the Elasticsearch 7 search engine.
          •     ci:test:search-db-partition - Test search functional tests with Database Partitioning.
          •     ci:test:search-environments - Test search tests with the Elasticsearch 7 search engine on available environment stacks.
          •     ci:test:search-remote - Test only search tests with the latest compatible remote Elasticsearch 7 version.
          •     ci:test:search-remote-minimum - Test search tests with the minimum compatible remote Elasticsearch 7 version.
          •     ci:test:search-solr - Test search tests with the Solr 8 search engine.
          •     ci:test:security - Test security tests.
          •     ci:test:seo - Test SEO tests.
          •     ci:test:setup-wizard - Test Setup Wizard tests.
          •     ci:test:sf - Test source formatter.
          •     ci:test:sharepoint - Test Sharepoint 2010 tests.
          •     ci:test:sharing - Test Asset Sharing tests.
          •     ci:test:site-initializer-liferaybotics - No description is available.
          •     ci:test:site-initializer-masterclass - No description is available.
          •     ci:test:site-initializer-raylife-ap - No description is available.
          •     ci:test:site-initializer-raylife-d2c - No description is available.
          •     ci:test:site-initializer-team-extranet - No description is available.
          •     ci:test:site-initializer-testray - No description is available.
          •     ci:test:site-template - Test Site Template tests.
          •     ci:test:smoke - Run smoke test with all available configurations in PortalSmoke.testcase
          •     ci:test:solutions-acceptance - Test Solutions acceptance tests.
          •     ci:test:stable - No description is available.
          •     ci:test:staging - Test Staging components tests.
          •     ci:test:staging-custom-db - No description is available.
          •     ci:test:startup - Test Portal startup.
          •     ci:test:sybase - Test functional and integration tests on Sybase database.
          •     ci:test:system - Test functional and integration tests for System Management (Virtual Instances, System Settings, Instance Settings)
          •     ci:test:tango - Test Tango (Segmentation and Personalization) team tests.
          •     ci:test:tango-acceptance - Test Tango (Segmentation and Personalization) team acceptance tests.
          •     ci:test:tango-analytics-cloud - Test segmentation and personalization with Analytics Cloud.
          •     ci:test:tck - Test TCK tests.
          •     ci:test:translation - Test Translation Management tests.
          •     ci:test:upgrade - Test upgrades against tomcat app server and all supported databases.
          •     ci:test:upgrade-all - Test upgrades from all components against all app servers and databases.
          •     ci:test:upgrade-app-servers - Test large database for upgrade performance regressions.
          •     ci:test:upgrade-client - Test upgrade client.
          •     ci:test:upgrade-performance - No description is available.
          •     ci:test:upstream-functional - Test upstream functional tests.
          •     ci:test:upstream-functional-db-partition - Test upstream functional tests with Database Partitioning.
          •     ci:test:users - Test Users and Organizations tests.
          •     ci:test:usm - Test USM tests.
          •     ci:test:usm-db-partition - Test USM functional tests with Database Partitioning.
          •     ci:test:usm-upgrade - Test USM upgrade tests.
          •     ci:test:vulcan-acceptance - Test headless team acceptance tests considering Vulcan and Commerce.
          •     ci:test:wiki - Test wiki tests.
          •     ci:test:workflow - Test Workflow tests.
          •     ci:test:workflow-es7-remote - Test Workflow tests with the latest compatible remote Elasticsearch 7 version.
          •     ci:test:workflow-metrics - Test Workflow metrics tests.

          @john-co
          Copy link

          john-co commented Feb 1, 2023

          ci:test:object:norebase

          @liferay-continuous-integration
          Copy link
          Collaborator

          The test will run without rebasing.

          @john-co
          Copy link

          john-co commented Feb 1, 2023

          ci:test:object

          @liferay-continuous-integration
          Copy link
          Collaborator

          ❌ ci:test:object - 196 out of 252 jobs passed in 2 hours 18 minutes

          Click 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:1387401_6113

          Base Branch:

          Branch Name: master
          Branch GIT ID: fe66b60dfb479e4bd8428c8bf9eb06d16116a157

          Upstream Comparison:

          Branch GIT ID: 9a39ceec5ccb53e151cd1513412be7199a63dc89
          Jenkins Build URL: EE Development Acceptance (master) - 4245 - 2023-01-31[07:39:11]

          ci:test:object - 196 out of 252 jobs PASSED

          56 Failed Jobs:

          196 Successful Jobs:
            For more details click here.

            Failures unique to this pull:

            1. ...
            Test bundle downloads:

            @liferay-continuous-integration
            Copy link
            Collaborator

            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.

            4 participants