Añadir restricción sobre las mínimas versiones de navegadores soportadas

[santiagorodriguez96]

Este PR añade una llamada a allow_browser en nuestro ApplicationController para permitir que únicamente se pueda usar la app con navegadores modernos.

[EmilioCristalli]

Usamos alguna funcionalidad que tengan solo los navegadores modernos y no andaría en los más viejos?
Si no es así, yo trataría de no bloquear navegadores más viejos si la aplicación va a andar bien

[santiagorodriguez96]

Usamos alguna funcionalidad que tengan solo los navegadores modernos y no andaría en los más viejos? Si no es así, yo trataría de no bloquear navegadores más viejos si la aplicación va a andar bien

Tiene sentido. Ahora mismo lo que veo que más nos limita es que el browser tenga support para importmaps: https://caniuse.com/import-maps – stimulus y turbo parecen estar soportados, según entiendo, por la mayoría de los browsers desde hace varias versiones atrás.

Por lo tanto yo iría simplemente con limitar los browsers a únicamente los que soportan importmaps:

allow_browser versions: { safari: 16.4, chrome: 89, firefox: 108, opera: 76, edge: 89, ie: false }

Qué te parece?

[EmilioCristalli]

Usamos alguna funcionalidad que tengan solo los navegadores modernos y no andaría en los más viejos? Si no es así, yo trataría de no bloquear navegadores más viejos si la aplicación va a andar bien

Tiene sentido. Ahora mismo lo que veo que más nos limita es que el browser tenga support para importmaps: https://caniuse.com/import-maps – stimulus y turbo parecen estar soportados, según entiendo, por la mayoría de los browsers desde hace varias versiones atrás.

Por lo tanto yo iría simplemente con limitar los browsers a únicamente los que soportan importmaps:

allow_browser versions: { safari: 16.4, chrome: 89, firefox: 108, opera: 76, edge: 89, ie: false }

Qué te parece?

Si el único problema es import-maps, yo votaría por agregar el shim (link). Creo que antes importmap-rails lo hacía automáticamente, pero a partir de la versión 2 ya no lo hace (PR)

[santiagorodriguez96]

Usamos alguna funcionalidad que tengan solo los navegadores modernos y no andaría en los más viejos? Si no es así, yo trataría de no bloquear navegadores más viejos si la aplicación va a andar bien

Tiene sentido. Ahora mismo lo que veo que más nos limita es que el browser tenga support para importmaps: https://caniuse.com/import-maps – stimulus y turbo parecen estar soportados, según entiendo, por la mayoría de los browsers desde hace varias versiones atrás.
Por lo tanto yo iría simplemente con limitar los browsers a únicamente los que soportan importmaps:

allow_browser versions: { safari: 16.4, chrome: 89, firefox: 108, opera: 76, edge: 89, ie: false }

Qué te parece?

Si el único problema es import-maps, yo votaría por agregar el shim (link). Creo que antes importmap-rails lo hacía automáticamente, pero a partir de la versión 2 ya no lo hace (PR)

En ese caso ya siento que no tengo muy en claro cuales versiones de browsers deberíamos soportar como mínimo; supongo que habría que hacer pruebas en navegadores viejos para empezar a ver en cuáles deja de andar la aplicación para decidir – lo cual a mi entender pasa a ser un esfuerzo considerable que dado el valor que nos puede llegar a dar siento que pierde un poco el sentido. A vos se te ocurre otra forma?

Qué te parece simplemente agregar el shim de importmaps cómo vos decís y no restringir las versiones de los browsers por ahora hasta que (en algún momento) tengamos más claridad sobre cuáles versiones no soportamos?

[EmilioCristalli]

Qué te parece simplemente agregar el shim de importmaps cómo vos decís y no restringir las versiones de los browsers por ahora hasta que (en algún momento) tengamos más claridad sobre cuáles versiones no soportamos?

a esto es lo que me refería con mi comentario anterior