Skip to content

testing final#3

Open
CesarDav wants to merge 1 commit intomasterfrom
feature/google-drive
Open

testing final#3
CesarDav wants to merge 1 commit intomasterfrom
feature/google-drive

Conversation

@CesarDav
Copy link

No description provided.

@CesarDav CesarDav requested a review from alacret July 13, 2019 13:12

//stripe
createPaymentRequest({data})
_sendEmail({data})
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
_sendEmail({data})
await _sendEmail({data})

}

// googledrive folder
await admin.firestore().collection('payments').doc(basicInformation.email).set({payamentData})
Copy link
Contributor

Choose a reason for hiding this comment

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

Cambia esto, deja que el ID lo autogenere firebase con .add, y coloca el email y la fecha como un campo mas, asi como lo tienes cada vez que vamos a escribir un payment nuevo hay que borrar el viejo

try {
let status = await stripe.charges.create({
amount: 1500,
amount: 1,
Copy link
Contributor

Choose a reason for hiding this comment

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

Esta valor no esta permitido por la API. @CesarDav acostumbrate por favor a leerte la documentacion para poder trabajar bien. Si te lees la documentacion de este metodo en Stripe te dice claramente cuales son los valores permitidos:
https://stripe.com/docs/api/charges/object

Copy link
Author

Choose a reason for hiding this comment

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

ok

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.

2 participants