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 102798 #82358

Closed
wants to merge 5 commits into from
Closed

Lps 102798 #82358

wants to merge 5 commits into from

Conversation

shuyangzhou
Copy link

No description provided.

pavel-savinov and others added 5 commits December 19, 2019 09:49
…segment experience id into account for non-instanceable portlets
…ossible segment experience id into account for non-instanceable portlets"

This reverts commit 45e43f5.
…ace in case when portlet is not instanceable"

This reverts commit 77f5f84.
@liferay-continuous-integration
Copy link
Collaborator

CI is automatically triggering "ci:test:sf" and "ci:test:relevant" for this pull to run Source Formatter and relevant tests.

Comment "ci:test" to run the full PR Tester for this pull.

@liferay-continuous-integration
Copy link
Collaborator

✔️ ci:test:sf - 1 out of 1 jobs passed in 2 minutes 52 seconds 987 ms

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 9629a5b9066669f06bfc3ad4cd5e5ccb902aa6c0

Sender Branch:

Branch Name: LPS-102798
Branch GIT ID: 4eeea0e086f616adcfd33df3b15b2ad54e9cdf90

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

@liferay-continuous-integration
Copy link
Collaborator

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

✔️ ci:test:relevant - 17 out of 17 jobs passed in 1 hour 28 minutes 23 seconds 871 ms

Click here for more details.

Base Branch:

Branch Name: master
Branch GIT ID: 5d166508a99c57e4c60aedc77027b949f91b8431

Copied in Private Modules Branch:

Branch Name: master-private
Branch GIT ID: f974066a1816a014c2c71d1641a953466a70ec53

ci:test:stable - 8 out of 8 jobs PASSED
8 Successful Jobs:
ci:test:relevant - 17 out of 17 jobs PASSED
17 Successful Jobs:
For more details click here.

@brianchandotcom
Copy link
Owner

Merged. Thank you.
View total diff: 6351f0b...1a41c65

@brianchandotcom
Copy link
Owner

@pavel-savinov fyi

@brianchandotcom
Copy link
Owner

@epgarcia see proper fix from @shuyangzhou

@epgarcia
Copy link

Thanks @brianchandotcom and @shuyangzhou for the review. However, I'm afraid that with your fix, the support for non-instanceable portlets in different page experiences was lost (see https://issues.liferay.com/browse/LPS-106351).

@arboliveira
Copy link

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.

6 participants