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

Fix try catch and static methods #45

Merged
merged 3 commits into from
Aug 25, 2017
Merged

Conversation

JoValo
Copy link
Contributor

@JoValo JoValo commented Aug 22, 2017

Presentaba el siguiente error cuando se ingresaba alguna tarjeta prueba que regresara un error:
pastedimage

Con el fix se trabaja correctamente el mensaje quedando de la siguiente forma:
captura de pantalla 2017-08-22 a la s 16 12 51

También se cambio la CDN de Conekta y se corrigio un Warning que lanzaba al actualizarla setPublishableKey is going to be deprecated on version 2.0.0

@@ -22,7 +22,7 @@
* @version v1.0.0
* @license http://opensource.org/licenses/osl-3.0.php Open Software License (OSL 3.0)
*}
<script type="text/javascript" src="http://ajax.googleapis.com/ajax/libs/jquery/1.11.1/jquery.min.js"></script>
<script type="text/javascript" src="https://ajax.googleapis.com/ajax/libs/jquery/1.11.1/jquery.min.js"></script>
<script type="text/javascript" src="https://conektaapi.s3.amazonaws.com/v0.3.2/js/conekta.js"></script>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

podrias cambiar conektaapi.s3.amazonws.com por https://cdn.conekta.io/js/latest/conekta.js

Copy link
Contributor Author

@JoValo JoValo Aug 22, 2017

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Se tokenizo con la nueva CDN
captura de pantalla 2017-08-22 a la s 17 11 31

@JoValo JoValo requested a review from fruizg0302 August 23, 2017 15:50
@JoValo JoValo merged commit 4f8012c into develop Aug 25, 2017
@JoValo JoValo deleted the feature/exception-correction branch September 9, 2017 05:11
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

Successfully merging this pull request may close these issues.

3 participants