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

Actualizar urls para método InSite #95

Closed
daparisi opened this issue Oct 23, 2023 · 8 comments
Closed

Actualizar urls para método InSite #95

daparisi opened this issue Oct 23, 2023 · 8 comments

Comments

@daparisi
Copy link

Hola,
Seria posible actualizar a las nuevas urls para el método InSite, según esta en la documentación oficial.
Parece que solo seria cambiar las urls en la función estática de getJsPath().

// Test

<script src="https://sis-t.redsys.es:25443/sis/NC/sandbox/redsysV3.js"></script>

// Real

<script src="https://sis.redsys.es/sis/NC/redsysV3.js"></script>

Saludos

@ssheduardo
Copy link
Owner

Si tienes a la mano el manual para echarle un vistazo.

@daparisi
Copy link
Author

I tanto!!

https://pagosonline.redsys.es/conexion-insite.html
En el apartado Creación del iFrame.

Incluso para que tenga compatibilidad con gente que use la versión 2, tal vez no estaría mal añadir un segundo parámetro a la función getJsPath() para poder indicar la versión del fichero a cargar.

Gracias.

@ssheduardo
Copy link
Owner

ssheduardo commented Oct 23, 2023

Si, esto estaba pensando.
Pero estoy viendo cuando agregarlo que un compañero esta haciendo un PR para sacar la V2 de la librería con compatibilidad desde la 8.1, 8.2, 8.3.
Pero con el código más alineado a dichas versiones, aunque actualmente esta librería es compatible desde PHP 7.4 creo hasta la 8.2 pero sin aplicar lo bueno que trae la 8.1 en adelante.

#94

@rogervila
Copy link
Contributor

Podemos añadir esta option en la 2.1 una vez haya salido la 2.0.

@ssheduardo
Copy link
Owner

Lo malo que le dejaremos a los que aún usan 7.4 o 8.0 sin ese cambio.

@rogervila
Copy link
Contributor

@ssheduardo entonces podrías incluir esta opción en la 1.5 y ya haré el merge en la 2.0. qué te parece?

@ssheduardo
Copy link
Owner

Si mejor, eso haré.

@ssheduardo
Copy link
Owner

Va ser la versión 1.4.6 dado que es una corrección menor.
Ya esta subido.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants