search

platanus / ng2-rut

Angular 2 library with several components to handle Chilean RUT validation, cleaning and formatting.
MIT License
29 stars 26 forks source link

Compatibilidad con Angular 9.1 #27

Closed mbaeza closed 4 years ago

mbaeza commented 4 years ago

En el momento que se utiliza con Angular 9.1, el validador de RUT no funciona debido a que no encuntra la librería Render. Sugiere utilizar la Librería Render2

tlarrain commented 4 years ago

Podrían mergear este PR y re compilar el paquete? gracias!

llekn commented 4 years ago

@gmq

skrosoft commented 4 years ago

Hello, alguien nos podría ayudar con esto? Se los agradecería un montón! @gmq @llekn @blackjid @ldlsegovia @niesteszeck @vigohe @v-karbovnichy

skrosoft commented 4 years ago

No estoy ultra metido en angular, pero por lo que veo no basta con cambiar a Renderer2. Adamas hay que cambiar las funciones que cambiaron en la nueva libreria.

me parece que solo es setElementProperty por setProperty https://angular.io/guide/migration-renderer

Efectivamente @blackjid , revisé detalladamente la documentación que indicas y cada uno de los métodos migrados pero si te fijas bien, el proyecto ya ocupa setElementProperty

https://github.com/platanus/ng2-rut/pull/27/files#diff-cb4765738f57bd0649c56f800787441eR32 / Línea 32

blackjid commented 4 years ago

Yo diria que es al revés. El metodo SetElementProperty es de Renderer no de Renderer2... image

skrosoft commented 4 years ago

Yo diria que es al revés. El metodo SetElementProperty es de Renderer no de Renderer2...

Tienes toda la razón, mis disculpas 😅

blackjid commented 4 years ago

@skrosoft quería pedirte disculpas, pero no vamos a mergear el PR.

Lo que pasa es que nosotros ya no usamos esta librería, porque dejamos de usar angular. No tiene mucho sentido que mantengamos una librería que no usamos, porque van a pasar estas cosas todo el rato.

Te invito a usar la libreria que hizo @danielsotopino https://github.com/platanus/ng2-rut/issues/25#issuecomment-604518731 https://github.com/landscapedotcl/ng9-rut

Vamos a poner un mensaje de deprecated a esta librería para no generar mas confusion y expectativas.

skrosoft commented 4 years ago

@blackjid se entiende perfectamente, no te preocupes. Muchas gracias por la recomendación y gracias por habernos informado