manuelgodoy / Bitely_Mobile

Mobile front end for Bitely
0 stars 0 forks source link

En la pantalla de editar perfil de usuario, los 4 ult digitos deberian aparecer #2

Closed manuelgodoy closed 8 years ago

manuelgodoy commented 9 years ago

En la pantalla de editar el perfil del usuario los ultimos 4 digitos, el tipo de tarjeta y la fecha de vencimiento deberian ser mostrados. Para esto deben agregarse mas datos al endpoint de /user (@manuelgodoy) y mostrarse esos resultados en la pantalla de edit profile.

gianko commented 9 years ago

Le estoy mandando a POST /User

stripeToken, brand (visa, master, etc), date (12/2016) y last4.

Y estoy asumiendo que el GET /User me va a dar esas mismas variables con _hascustomertoken en vez del stripeToken.

Como bonus, sería ideal que GET /user me devolviera otras variables que estoy usando en la app y que seguro vds necesitan también. El avatar por ejemplo, y eso también disminuiría una llamada a facebook.

manuelgodoy commented 9 years ago

Tu no tienes que mandar esos datos, solo manda el stripeToken, nosotros te mandamos esos datos. Lo del avatar lo podemos hacer. On Sep 6, 2015 7:08 PM, "Gian" notifications@github.com wrote:

Le estoy mandando a POST /User

stripeToken, brand (visa, master, etc), date (12/2016) y last4.

Y estoy asumiendo que el GET /User me va a dar esas mismas variables con _hascustomertoken en vez del stripeToken.

Como bonus, sería ideal que GET /user me devolviera otras variables que estoy usando en la app y que seguro vds necesitan también. El avatar por ejemplo, y eso también disminuiría una llamada a facebook.

— Reply to this email directly or view it on GitHub https://github.com/manuelgodoy/Bitely_Mobile/issues/2#issuecomment-138139289 .

manuelgodoy commented 9 years ago

La idea es que tu no guardes esos datos. Nosotros te los vamos a mandar. Creo que no puedes cerrar este issue haata que nosotros no lo hagamos. On Sep 6, 2015 7:11 PM, "Manuel Godoy" manuelgm@gmail.com wrote:

Tu no tienes que mandar esos datos, solo manda el stripeToken, nosotros te mandamos esos datos. Lo del avatar lo podemos hacer. On Sep 6, 2015 7:08 PM, "Gian" notifications@github.com wrote:

Le estoy mandando a POST /User

stripeToken, brand (visa, master, etc), date (12/2016) y last4.

Y estoy asumiendo que el GET /User me va a dar esas mismas variables con _hascustomertoken en vez del stripeToken.

Como bonus, sería ideal que GET /user me devolviera otras variables que estoy usando en la app y que seguro vds necesitan también. El avatar por ejemplo, y eso también disminuiría una llamada a facebook.

— Reply to this email directly or view it on GitHub https://github.com/manuelgodoy/Bitely_Mobile/issues/2#issuecomment-138139289 .

gianko commented 9 years ago

Jajaja... no sabía que "Fixes" lo cerraba, creí que solo "Close" lo cerraba, ya lo abro.

El response con el token me da esos datos, no tengo problema en enviarselos para que no llamen vds a Stripe otra vez.

Yo guardo es el response de /user

manuelgodoy commented 9 years ago

OK el /user endpoint fue editado para incluir datos de la tarjeta de credito: El Response en este endpoint ahora es asi:

{u'user': {u'email': u'DavidBlalock@hotmail.com',
           u'fullname': u'David Blalock',
           u'has_customertoken': True,
           u'nickname': u'DavidBlalock',
           u'stripe_data': {u'account_balance': 0,
                            u'created': 1442287032,
                            u'currency': None,
                            u'default_source': u'card_16l9wSHBGnM76RwSMQWKwkAW',
                            u'delinquent': False,
                            u'description': u'Creating customer: DavidBlalock',
                            u'discount': None,
                            u'email': u'DavidBlalock@hotmail.com',
                            u'id': u'cus_6z8CFjLxhbfLFQ',
                            u'livemode': False,
                            u'metadata': {},
                            u'object': u'customer',
                            u'shipping': None,
                            u'sources': {u'data': [{u'address_city': None,
                                                    u'address_country': None,
                                                    u'address_line1': None,
                                                    u'address_line1_check': None,
                                                    u'address_line2': None,
                                                    u'address_state': None,
                                                    u'address_zip': None,
                                                    u'address_zip_check': None,
                                                    u'brand': u'Visa',
                                                    u'country': u'US',
                                                    u'customer': u'cus_6z8CFjLxhbfLFQ',
                                                    u'cvc_check': u'pass',
                                                    u'dynamic_last4': None,
                                                    u'exp_month': 12,
                                                    u'exp_year': 2016,
                                                    u'fingerprint': u'bDiCuhvcF8ZvI3cs',
                                                    u'funding': u'credit',
                                                    u'id': u'card_16l9wSHBGnM76RwSMQWKwkAW',
                                                    u'last4': u'4242',
                                                    u'metadata': {},
                                                    u'name': None,
                                                    u'object': u'card',
                                                    u'tokenization_method': None}],
                                         u'has_more': False,
                                         u'object': u'list',
                                         u'total_count': 1,
                                         u'url': u'/v1/customers/cus_6z8CFjLxhbfLFQ/sources'},
                            u'subscriptions': {u'data': [],
                                               u'has_more': False,
                                               u'object': u'list',
                                               u'total_count': 0,
                                               u'url': u'/v1/customers/cus_6z8CFjLxhbfLFQ/subscriptions'}}}}

Para llegarle a los 4 ultimos digitos tendrias que hacer response['user']['stripe_data']['sources']['data']['last4']

En la pantalla de profile llegandole desde el side menu seria bueno ensenar esta informacion por los momentos.

gianko commented 9 years ago

Y yo tengo que enviarles estos datos?

mi /user no me devuelve eso :cry:

manuelgodoy commented 9 years ago

No no, tu no los envias. Acuerdate que tu no envias nada en Stripe solo el token nosotros nos encargamos de todo. No lo actualice mala mia.

gianko commented 9 years ago

Listo... para la versión 2 podemos mejorar esto. ahora estoy haciendo 2 llamados

manuelgodoy commented 9 years ago

Por que 2 llamados?

gianko commented 9 years ago

son 3 de hecho...

uno a facebook/me otro a /facebook_login_app y uno a /user

manuelgodoy commented 9 years ago

mm OK, pero eso solo en esta pantalla no?

gianko commented 9 years ago

solo durante el login

manuelgodoy commented 9 years ago

Quieres que te mande toda la info del usuario con /facebook_login_app ?

gianko commented 9 years ago

para el proximo update, si no tengo que cambiar varias cosas...

manuelgodoy commented 9 years ago

Funciona que se lean los ult 4 digitos. Pero tambien deberia haber una forma para llenar los datos de la tarjeta en el Profile si el usuario no tiene la tarjeta en el sistema

gianko commented 9 years ago

Esto está, pero no hay link para llegarle. On Sep 17, 2015 2:15 AM, "Manuel Godoy" notifications@github.com wrote:

Funciona que se lean los ult 4 digitos. Pero tambien deberia haber una forma para llenar los datos de la tarjeta en el Profile si el usuario no tiene la tarjeta en el sistema

— Reply to this email directly or view it on GitHub https://github.com/manuelgodoy/Bitely_Mobile/issues/2#issuecomment-140984483 .

manuelgodoy commented 9 years ago

Como que no hay link? Tu halas el.user desde profile, si no tiene last4 enseñas la forma para llenar la tarjeta. On Sep 17, 2015 5:45 AM, "Gian" notifications@github.com wrote:

Esto está, pero no hay link para llegarle. On Sep 17, 2015 2:15 AM, "Manuel Godoy" notifications@github.com wrote:

Funciona que se lean los ult 4 digitos. Pero tambien deberia haber una forma para llenar los datos de la tarjeta en el Profile si el usuario no tiene la tarjeta en el sistema

— Reply to this email directly or view it on GitHub < https://github.com/manuelgodoy/Bitely_Mobile/issues/2#issuecomment-140984483

.

— Reply to this email directly or view it on GitHub https://github.com/manuelgodoy/Bitely_Mobile/issues/2#issuecomment-141040728 .

gianko commented 9 years ago

Porque estaba la pantalla de editar perfil y la de editar tarjeta. Pero en el side menu no aparecen ni en ningun otro lado.

Hay varios detalles de esos diseño que acomodar.

2015-09-17 10:50 GMT-04:30 Manuel Godoy notifications@github.com:

Como que no hay link? Tu halas el.user desde profile, si no tiene last4 enseñas la forma para llenar la tarjeta. On Sep 17, 2015 5:45 AM, "Gian" notifications@github.com wrote:

Esto está, pero no hay link para llegarle. On Sep 17, 2015 2:15 AM, "Manuel Godoy" notifications@github.com wrote:

Funciona que se lean los ult 4 digitos. Pero tambien deberia haber una forma para llenar los datos de la tarjeta en el Profile si el usuario no tiene la tarjeta en el sistema

— Reply to this email directly or view it on GitHub <

https://github.com/manuelgodoy/Bitely_Mobile/issues/2#issuecomment-140984483

.

— Reply to this email directly or view it on GitHub < https://github.com/manuelgodoy/Bitely_Mobile/issues/2#issuecomment-141040728

.

— Reply to this email directly or view it on GitHub https://github.com/manuelgodoy/Bitely_Mobile/issues/2#issuecomment-141120016 .

manuelgodoy commented 9 years ago

Entiendo. Bueno habria que poner la forma en Profile. Despues arreglamos con mas detalle

gianko commented 9 years ago

Listo