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

Deprecated #19

Open
carlosmjr opened this issue Jan 31, 2017 · 15 comments
Open

Deprecated #19

carlosmjr opened this issue Jan 31, 2017 · 15 comments

Comments

@carlosmjr
Copy link

Os adjunto un aviso, para que lo cambiéis:
DEPRECATION WARNING: skip_before_filter is deprecated and will be removed in Rails 5.1. Use skip_before_action instead. (called from class:TpvController at //ruby-2.3.3/gems/redsys-rails-0.0.6/app/controllers/redsys/tpv_controller.rb:3)

Aún no he conseguido que me funcione la gema, me estoy volviendo loco, pero si veo algo os aviso.

@mac-manu
Copy link

mac-manu commented Aug 3, 2018

skip_before_filter' for Redsys::TpvController:Class Did you mean? skip_before_action
lo he cambiado , pero habra que esperar a que se "comittee"

@makinox
Copy link

makinox commented Dec 27, 2018

Joder, un año y nada? Que falta de respeto, ya en rails 5.1 esto no funciona.

@mac-manu
Copy link

creo que este proyecto fue abandonado @makinox
habra que ir pensando en hacer un fork y seguir ...

@makinox
Copy link

makinox commented Dec 27, 2018

Tocará, de igual manera me parece muy mal teniendo en cuenta que esta es la librería oficial de redsys y que se está pagando por este servicio.
Con razón soluciones como stripe, payU, Google pay, etc; Crecen tanto, me llevo una muy mala impresión de este servicio, no lo volveré a utilizar.

@Senen
Copy link
Member

Senen commented Dec 27, 2018

Hola @makinox, como dice @mac-manu lo del fork me parece una idea fantástica! Nosotros no recibimos nada de Redsys (hace tiempo contactamos con Redsys por si querían sponsorizar el mantenimiento de la gema pero no nos hicieron ni caso) por lo que no tenemos muchos motivos para seguir con esta gema actualmente!

No obstante os pido disculpas @carlosmjr , @mac-manu , @makinox por no haberos respondido antes. Cuando tenga un rato o bien la arreglo , o por el contraro añadiré al Readme una nota bien grande que ponga Unmaintained.

@Senen
Copy link
Member

Senen commented Dec 27, 2018

@makinox de oficial nada de nada! 😄

@mac-manu
Copy link

Tocará, de igual manera me parece muy mal teniendo en cuenta que esta es la librería oficial de redsys y que se está pagando por este servicio.
Con razón soluciones como stripe, payU, Google pay, etc; Crecen tanto, me llevo una muy mala impresión de este servicio, no lo volveré a utilizar.

@makinox el tema es que si "el cliente" lo requiere, ahora propondremos con mas fuerza otros métodos de pago...

@mac-manu
Copy link

@makinox de oficial nada de nada! 😄

si no pagan ... de oficial nada.

@taitus
Copy link
Member

taitus commented Dec 27, 2018

@makinox Donde has visto que es la "librería oficial de redsys"? solo por curiosidad. Gracias

@makinox
Copy link

makinox commented Dec 27, 2018

@taitus Mi empresa como que compró el servicio y me pasaron los documentos de integración, ahí mencionaban las API que en las cuales se encontraba está para rails.

@taitus
Copy link
Member

taitus commented Dec 27, 2018

@makinox Gracias por la información, es la primera noticia que tenemos de que esta gema está considerada como "librería oficial de Redsys".

@Senen
Copy link
Member

Senen commented Dec 27, 2018

¿Podrías pasarnos ese documento @makinox? Es para volver a contactar con Redsys y aclarar esta situación. Ahora lo único que puedo pensar es que Redsys a agregado nuestra libreía a su documentación sin preguntarnos ni respondernos al email .... lo que desde luego me parece muy mal por parte de Redsys.

Voy a aclarar un par de puntos que me parece que están confundiendo a la gente:

  1. Nosotros no tenemos nada que ver con Redsys (no trabajamos ni hemos trabajado para ellos)
  2. Esta gema la creamos hace mucho tiempo para un cliente que se ha quedado en rails 4 y por eso no la seguimos actualizando

Espero haberte aclarado algo @makinox, y ya que veo que este tema te está molestando notablemente te recomiendo que te crees un fork y que te arregles el código que no te funcione.

@makinox
Copy link

makinox commented Dec 27, 2018

No en realidad no estoy molesto jajaja, tal vez mi tono se sobre entendió, no hay problema, ya encontré una solución.
Ahora bien creo que no te podré pasar ese documento por problemas legales, gracias por las respuestas y el soporte, espero que todos tengan unas felices fiestas 🍰.

@Senen
Copy link
Member

Senen commented Dec 27, 2018

Me alegro! Igualmente @makinox! 😄

Ya he añadido la nota al README para evitar esta confusión en un futuro! Gracias a todos por vuestros comentarios y lamento que la gema no esté mantenida! Volveremos a hablar con Redsys y si finalmente no la quiere mantener, la eliminaremos. 😞

@carlosmjr
Copy link
Author

Hola @makinox, como dice @mac-manu lo del fork me parece una idea fantástica! Nosotros no recibimos nada de Redsys (hace tiempo contactamos con Redsys por si querían sponsorizar el mantenimiento de la gema pero no nos hicieron ni caso) por lo que no tenemos muchos motivos para seguir con esta gema actualmente!

No obstante os pido disculpas @carlosmjr , @mac-manu , @makinox por no haberos respondido antes. Cuando tenga un rato o bien la arreglo , o por el contraro añadiré al Readme una nota bien grande que ponga Unmaintained.

Ya te digo yo, que no quiero tus disculpas. No son necesarias.
Al final usé otra gema, pero el trastear ésta me ayudó mucho a aprender algunas cosas.
Gracias por tu esfuerzo.

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

5 participants