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

Warnings de dependencias #1118

Closed
Tracked by #1102 ...
tfloxolodeiro opened this issue Sep 16, 2022 · 3 comments
Closed
Tracked by #1102 ...

Warnings de dependencias #1118

tfloxolodeiro opened this issue Sep 16, 2022 · 3 comments
Assignees

Comments

@tfloxolodeiro
Copy link
Contributor

Al subir a la version v3.24 de ember, se deprecaron tryInvoke, getWithDefault y otras features de ember, las cuales son usadas por algunas de nuestras dependencias, particularmente ember-paper. Esto hace que se tiren muchos warnings molestos, especialmente al correr tests.

Una forma de resolver varios de estos warnings, es actualizando las dependencias de ember-paper. Para hacer esto podriamos hacer un fork de ember-paper (de nuevo), lo cual vendria bien porque la ultima version de ember-paper usa getWithDefault (deprecado), lo cual no se puede resolver subiendo versiones de dependencias.
Tambien podriamos subir la version de node de pilas bloques a gallium para poder usar el override en package.json para forzar que ember-paper use dependencias mas actualizadas. Esto lo intentamos asi nomas pero explota, asi que parece que hay que meterle un poco de cerebro para hacerlo. Aun asi, esto no resolveria lo de que ember-paper usa getWithDefault.
Tirar un PR a ember-paper no es una opcion porque esta muy muerto :).

Tambien habria que actualizar la dependencia ember-composable-helpers a la version 5.0.0, pero esta dependencia solo causa unos pocos warnings, relativo al total de la pared de texto gigante de warnings totales, asi que es algo menor.

Tambien hay otros warnings que no vimos en profundidad porque no pudimos resolver lo de ember-paper.

Sufrimiento.

@tfloxolodeiro tfloxolodeiro added dependencies Pull requests that update a dependency file medium labels Sep 16, 2022
@asanzo
Copy link
Contributor

asanzo commented Oct 4, 2022

adopted-ember-addons/ember-paper#1202

Issue en ember paper sobre ember 4.

Entonces tenemos que seguir por el plan del fork. Investiguemos por ahí: actualizarle las dependencias a paper.

Quizás se pueden revisar los PR que hoy tiene ember-paper a ver si alguno soluciona alguno de los problemas.

@dlopezalvas
Copy link
Contributor

dlopezalvas commented Oct 25, 2022

Quedan:

  • deprecated this.render de nuestro lado en:
    https://github.com/Program-AR/pilas-bloques/blob/68f8146ca04fba99efc47c3190bc10219f842133/tests/helpers/actividadTest.js#L188-L200
    imagen

  • {"type":"warn","text":"WARNING: Binding style attributes may introduce cross-site scripting vulnerabilities; please ensure that values being bound are properly escaped. For more information, including how to disable this warning, see https://emberjs.com/deprecations/v1.x/#toc_binding-style-attributes. Style affected: \"position:fixed;\""}

  • {"type":"warn","text":"DEPRECATION: loc is deprecated, please use a dedicated localization solution like ember-intl. More alternatives listed at https://emberobserver.com/categories/internationalization. [deprecation id: ember-string.loc] See https://deprecations.emberjs.com/v3.x#toc_ember-string-loc for more details.\n registerHandler/HANDLERS[type]@http://localhost:7357/assets/vendor.js:54848:17\n raiseOnDeprecation@http://localhost:7357/assets/vendor.js:54742:13\n registerHandler/HANDLERS[type]@http://localhost:7357/assets/vendor.js:54848:17\n @http://localhost:7357/assets/test-support.js:23818:10\n

This was referenced Dec 5, 2022
@tfloxolodeiro tfloxolodeiro added ember-upgrade and removed dependencies Pull requests that update a dependency file medium labels Dec 5, 2022
@asanzo
Copy link
Contributor

asanzo commented Jan 31, 2024

imagen
migraremos, migraremos 🎵

@asanzo asanzo closed this as completed Jan 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

3 participants