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

COMMERCE-2297 Fix category display page editing #1599

Closed
wants to merge 2 commits into from

Conversation

ethanbustad
Copy link
Contributor

https://issues.liferay.com/browse/COMMERCE-2297

Let me know what you think of this fix -- not sure this is the best way, but it takes such a minor change that I can't pass it up :)

@liferay-continuous-integration
Copy link
Member

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.

@ethanbustad
Copy link
Contributor Author

ci:test

@liferay-continuous-integration
Copy link
Member

Validation PASSED. Running batch tests.

Build Time: 21 minutes 39 seconds 769 ms

Base Branch:

Branch Name: 7.1.x
Branch GIT ID: fcdbb7a76849c112f5888f4e0b9f71d1beae6a95

Task Summary:
  • subrepository-compile-jdk8 - ✔️
  • subrepository-semantic-versioning-jdk8 - ✔️
  • subrepository-pmd-jdk8 - ✔️
  • subrepository-source-format-jdk8 - ✔️
  • central-requirements-jdk8/subrepository-unit-jdk8 - ✔️
For full console, click here.

Test Results:

10 Tests Passed.
0 Tests Failed.

For all test results, click here.

@liferay-continuous-integration
Copy link
Member

@marco-leo
Copy link
Member

forwarded to #1717

@marco-leo marco-leo closed this Nov 29, 2019
@marco-leo
Copy link
Member

@ethanbustad please see the changes as the groupid is wrong

@ethanbustad ethanbustad deleted the COMMERCE-2297 branch January 31, 2020 19:04
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.

3 participants