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-99882 Remove empty string from character blacklist, as every string contain the empty string, it would make creation of journal folder impossible #2507

Closed

Conversation

victorg1991
Copy link

No description provided.

jbalsas and others added 30 commits August 14, 2019 16:09
…ay.getURLCurrent(). getURLCurrent() only works when calling from the displayContext because the url is the correct one, calling it from the resourceCommand retrieve a invalid url
…ulate the url from the resourceCommand. Remove some query param that are not necessary and set the selectedPanel
Update dependencies to bring in:

liferay/liferay-js-toolkit#380

This means that from now on, we will perform dead code elimination
during bundling, which means that modules like frontend-js-react-web
will now include only development code, or only production code,
respecting the setting of nodejs.node.env in the build.properties. This
will reduce bundle sizes, improve the debugging experience, and
eliminate some ugly errors in the browser console.

The eslint-config-liferay update activates one new rule, which required
a lint fix in item-selector-taglib.
timpak and others added 26 commits August 15, 2019 20:20
…ng contain the empty string, it would make creation of journal folder impossible
@liferay-continuous-integration
Copy link
Collaborator

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.

@ealonso
Copy link
Owner

ealonso commented Aug 19, 2019

Just started reviewing :)

:octocat: Sent from GH.

@ealonso ealonso closed this Aug 19, 2019
@victorg1991 victorg1991 deleted the LPS-99882_blacklist_folder branch October 29, 2019 07:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet