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

deps: ngx-custom-validators instead of ng2-validations #296

Merged

Conversation

liquid-tadan
Copy link
Contributor

PR Type

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] CI-related changes
[ ] Documentation content changes
[ ] Application / infrastructure changes
[ ] Other:

What Is the Current Behavior?

While upgrading to Angular 9 within the PWA upgrade to 0.20 I faced the following bug while running npm i

Symbol Base64Validator declared in /node_modules/ng2-validation/dist/base64/directive.js is not exported from /node_modules/ng2-validation/dist/base64/directive (import into /node_modules/ng2-validation/dist/index.js)

Issue Number: Closes #286

What Is the New Behavior?

Install the new dependency and change the import of ng2-validation to ngx-custom-validators, the latter has been forked from the original one and supports Angular 9.

Does this PR Introduce a Breaking Change?

[ ] Yes
[x] No

Other Information

@shauke shauke self-requested a review June 26, 2020 06:39
@shauke shauke self-assigned this Jun 26, 2020
@shauke shauke added the dependencies Pull requests that update a dependency file label Jun 26, 2020
@shauke shauke added this to the 0.22 milestone Jun 26, 2020
@shauke
Copy link
Collaborator

shauke commented Jun 26, 2020

Thank you very much for your contribution!
I will review your changes in the next days and let you know if everything works as expected.

@shauke shauke merged commit de6c3c9 into intershop:develop Jun 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

package ng2-validation is deprecated
2 participants