-
Notifications
You must be signed in to change notification settings - Fork 138
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
COMMERCE-1621 Liferay 7.2 compatibility!! #1373
Conversation
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. |
ci:test |
Validation FAILED.Build Time: 11 minutes 5 seconds 94 ms Base Branch:Branch Name: 7.1.x Task Summary:
For full console, click here. |
07967dd
to
0b0384f
Compare
I think this won't pass until the changes I made to 7.1 are published, which apparently hasn't been done yet.... |
5202235
to
78deb35
Compare
ci:test |
Validation FAILED.Build Time: 24 minutes 1 second 542 ms Base Branch:Branch Name: 7.1.x Task Summary:
For full console, click here. |
All of these changes also work in 7.1
78deb35
to
29625c8
Compare
ci:test |
Validation PASSED. Running batch tests.Build Time: 17 minutes 50 seconds 554 ms Base Branch:Branch Name: 7.1.x Task Summary:
For full console, click here. |
https://issues.liferay.com/browse/COMMERCE-1621
This works on 7.1.
This works on 7.2 except for a couple frontend issues.